You're correct. Although I don't think the slot gets disconnected when a NULL model would be set later on, so it's not completely impossible to reach here with model being NULL.

If there's anything I've learned in years of writing C++ code it's that there is no such thing as too many NULL pointer checks. Better to be safe than sorry, please just keep it :)