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

Reply via email to