Possible memory leak in rlm_sql?
Alan DeKok
aland at deployingradius.com
Fri Dec 17 15:32:26 CET 2010
Brian Candler wrote:
> I noticed something in rlm_sql.c function rlm_sql_process_groups().
>
> group_list is allocated at the top of the function, but
> sql_grouplist_free(&group_list) is only called at the end. All the various
> error exits don't call it.
>
> ISTM that's going to leak memory in event of errors, but perhaps I have
> overlooked something which prevents that.
Nope. You're right.
Alan DeKok.
More information about the Freeradius-Users
mailing list