[PATCH] Fix CID 1415704 Integer overflowed argument

Andreas Schneider asn at samba.org
Mon Aug 7 08:02:09 UTC 2017


On Friday, 4 August 2017 14:59:45 CEST Andrej Gessel via samba-technical 
wrote:
> Hi,
> 
> i've looked through last commits on master and maybe found some mistake.
> Which function should be really used here?
 Commit message: strtoul
> Commit: strtol
> 
> I think strtol, because variable "n", that holds return value of function,
> is of type long.


Yes, the commit message is wrong. strtol is correct.


Thanks for reviewing!


	Andreas

-- 
Andreas Schneider                   GPG-ID: CC014E3D
Samba Team                             asn at samba.org
www.samba.org



More information about the samba-technical mailing list