Re: [Ipmitool-devel] multiple uint8_t overflow in 'lib/ipmi_main.c' viaparameters

2011-10-24 Thread Zdenek Styblik
Thank you for your comments. On Mon, Oct 24, 2011 at 9:11 PM, Andy Cress wrote: > > The problem scenario is that a user enters a string with a number that > has invalid data or is too large for uint8_t.  The result is that the > current code would then use LONG_MAX or LONG_MIN (truncated to one >

Re: [Ipmitool-devel] multiple uint8_t overflow in 'lib/ipmi_main.c' viaparameters

2011-10-24 Thread Andy Cress
The problem scenario is that a user enters a string with a number that has invalid data or is too large for uint8_t. The result is that the current code would then use LONG_MAX or LONG_MIN (truncated to one byte). Not a big problem, but an error message would be better. Comments on the routin