Z, The patch looks good to me. It certainly fixes the reported problem. -- Jim Mankovich | jm...@hp.com (MST) --
On 3/8/2013 1:45 PM, Zdenek Styblik wrote: > Hello all, > > attached is a diff of proposed change to 'lib/ipmi_lanp.c'. This bug > has been reported by Ales Ledvinka a the problem is this ``if (p == > NULL)'' condition is never met, because struct is on heap, if I > understood and writing it correctly. > Comments? Ideas? I've tested it on small snippet code and it seems to > work just fine. > It's possible such change will be required at more places in > 'lib/ipmi_lanp.c'. > > Thanks, > Z. ------------------------------------------------------------------------------ Symantec Endpoint Protection 12 positioned as A LEADER in The Forrester Wave(TM): Endpoint Security, Q1 2013 and "remains a good choice" in the endpoint security space. For insight on selecting the right partner to tackle endpoint security challenges, access the full report. http://p.sf.net/sfu/symantec-dev2dev _______________________________________________ Ipmitool-devel mailing list Ipmitool-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ipmitool-devel