[PATCH] Fix string to integer conversion

swen swen at linux.ibm.com
Fri Jan 25 17:45:39 UTC 2019


On Fri, 2019-01-25 at 08:59 -0800, Jeremy Allison wrote:
> On Fri, Jan 25, 2019 at 09:30:02AM +0100, swen via samba-technical
> wrote:
> > All
> > 
> > There's a merge request for this small patch-set on gitlab
> > https://gitlab.com/samba-team/samba/merge_requests/197
> > with all tests passed !
> > 
> > Just in case someone prefers to look at it seperatly 
> > please have a look at the attachment.
> > 
> > Please review and push if happy.
> 
> NAK on this patch for the reasons you and I already
> discussed. You haven't addressed any of them and I
> will continue to object to this until you do.
> 
Jeremy

this is the modified first version of the patchset to which
you commented

> 1) nttime_from_string() is actually an unused function.
>    ...
>    Rather than fix, just delete it !

=> I did that 

> 2) Setting errno=0 inside VFS code is fraught with
> danger. Your changes indirectly do this by changing
> conv_str_size() to set errno to zero as a side effect
> which is called by some of the VFS functions.

=> I handled that exactly as you suggested by saving
   the original errno value and restoring it again 
   after strtoull got dealed with.

The only thing the 2 of us couldn't get a common opinion on
is how a possible wrapper for the above could look like.
Regarding that you said in another mail.

> I am not the ultimate gatekeeper on this, if you
> can find 2 other Samba Team engineers to agree
> with you, you can override my veto.

... and I thought, that's what I'm trying to do here.

Did I misunderstand something ?

Cheers Swen

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20190125/f91c11d2/signature.sig>


More information about the samba-technical mailing list