[STABLE] Small radius.c patch

Russ Allbery rra at stanford.edu
Wed Jul 4 05:21:36 UTC 2001


Katsuhiro Kondou <Katsuhiro_Kondou at isc.org> writes:
> Bettina Fink <laura at hydrophil.de> wrote;

>> Our "code hog", which gave me the patch (I personally never looked into
>> the radius stuff), says "Yes", the accompanying text was "Fixes
>> (possible) memory freeing problem, at least on FreeBSD".

> radius_config is never free'ed in the code.  Or am I missing something?

Hm.  Yeah, you're right....

I think that the COPY is cleaner, since otherwise one is using the static
return of cpcatpath, but I guess it isn't actually used for much so it
doesn't matter at all.  When I switched it to concatpath, I added a free.

-- 
Russ Allbery (rra at stanford.edu)             <http://www.eyrie.org/~eagle/>


More information about the inn-patches mailing list