[PATCH 4/6] libsensors4: Use strtoul

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jean Delvare wrote:
> Hi Hans,
> 
> On Wed, 15 Aug 2007 18:59:25 +0200, Hans de Goede wrote:
>> Jean Delvare wrote:
>>> Use strtoul() instead of parsing integers on our own.
>> Erm, why all the < 0 checks on the return value of strtoul, are the variable in 
>> which the return value gets stored signed and can we have a wrap?
> 
> Yes, this is the reason why I added these "< 0" checks.
> 
>> In that case wouldn't it be better to first store in an unsigned long and then 
>> explicitly check the limits we want to enforce?
> 
> There's no specific limit I want to enforce, I only want to make sure
> that we don't end up with a negative value due to a possible wrap. This
> is pure paranoia, BTW, no sane user would ever trigger it. I agree that
> letting the overflow happen and checking for it afterwards isn't very
> elegant, but it works and it is efficient, so I am a bit reluctant to
> make the code more complex than it needs be.
> 
> I could probably use strtol instead of strtoul if you prefer, though.
> 

No its fine as is then.

Regards,

Hans





[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux