Re: test null before free in relayd

2015-05-27 Thread sven falempin
Unmangled diff: http://pastebin.com/p9Qi9rZX On Mon, May 25, 2015 at 4:24 PM, sven falempin wrote: > > > On Mon, May 25, 2015 at 2:55 PM, Gleydson Soares > wrote: >> >> >> your diff is incomplete... >> well, why just this occurrence? there is others check null before >> free(3) in same file. >

Re: test null before free in relayd

2015-05-25 Thread sven falempin
On Mon, May 25, 2015 at 2:55 PM, Gleydson Soares wrote: > > > your diff is incomplete... > well, why just this occurrence? there is others check null before > free(3) in same file. > > and please, be more specific adding more details about your changes: eg.: > (I am removing the null check here, b

Re: test null before free in relayd

2015-05-25 Thread Gleydson Soares
On Mon, May 25, 2015 at 1:23 PM, sven falempin wrote: > Index: relay.c > === > RCS file: /cvs/src/usr.sbin/relayd/relay.c,v > retrieving revision 1.194 > diff -u -p -r1.194 relay.c > --- relay.c 18 May 2015 16:57:20 - 1.1

test null before free in relayd

2015-05-25 Thread sven falempin
Index: relay.c === RCS file: /cvs/src/usr.sbin/relayd/relay.c,v retrieving revision 1.194 diff -u -p -r1.194 relay.c --- relay.c 18 May 2015 16:57:20 - 1.194 +++ relay.c 25 May 2015 16:20:16 - @@ -1649,8 +1649,7 @@