Not much point supporting `GetFreeDockingPort(nullptr)` calls if we're not going to use them. Better to `assert(s);` at the beginning of the function and remove this check.