On 11/12/17(Mon) 15:46, kshe wrote: > On Sun, 10 Dec 2017 11:25:50 +0000, Martin Pieuchot wrote: > > On 08/12/17(Fri) 12:58, kshe wrote: > > > I noticed one instance where the size given to free(9) can easily be > > > determined. > > > > What about the other free(9)s in the same function? > > Somehow I did not immediately realize that rtm_report() would simply > store the allocated length in rtm->rtm_msglen. Now that I do, here is > an updated diff dealing with the remaining calls (along with some more > unrelated whitespace fixes).
Thanks. I'd suggest you for the next time to not to mix withespace or style changes with a functional change. That said it'd be great if you could look at other free(9) calls missing the size argument. Cheers, Martin