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

[Alan DeKok]

On Apr 15, 2020, at 8:35 AM, Chaigneau, Nicolas via Freeradius-Devel <freeradius-devel at lists.freeradius.org> wrote
> 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

  I've pushed a fix, thanks.

  Alan DeKok.