[4.0.x] in non-developer build, _fr_assert_fail logs two messages if called with "msg"

[Chaigneau, Nicolas]

Hello,



In non-developer build, if function _fr_assert_fail is called with a non NULL "msg" argument, then we get two messages printed (fr_fault_log is called twice).
This is probably not the expected behaviour.

Maybe add a "else" here?

https://github.com/FreeRADIUS/freeradius-server/blob/master/src/lib/util/debug.c#L1344



Regards,
Nicolas.

This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.