Validate before allocating WFIFO buffer, not after

[MrKeiKun]

Pull Request Prelude

• I have followed proper Hercules code styling.
• I have read and understood the contribution guidelines before making this PR.
• I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.

Changes Proposed

Several functions called WFIFOHEAD before finishing validation/data-gathering, then returned without calling WFIFOSET. Reordered so checks happen first and the buffer is only allocated once a packet is actually going to be sent:

• chrif_save_scdata: count active status changes before allocating.
• chrif_update_ip: check whether the IP changed before allocating.
• clif_updatestatus: reject an unrecognized type before allocating (added a validation switch mirroring the real switch's case labels, since the valid values aren't a contiguous range).
• clif_use_card: count insertable items before allocating.
• clif_sendegg: count hatchable pet eggs before allocating.

Issues addressed: #1041