I believe this is currently dead code, since we only `break` above if `packet` is NULL anyway.

I see the argument to leave this here anyway, though.  Changes in the above block could introduce other error situations that would require freeing the packet.