We *MUST NOT* free the data pointer explicitly. Anything that allocates and expects it to be freed MUST allocate from the context of the packet.

This caused abort() in radsniff
We have to free packets properly with rad_free instead of just free