[PATCH] Fix string to integer conversion
Ralph Böhme
slow at samba.org
Thu Feb 28 09:33:25 UTC 2019
Hi Swen,
On Tue, Feb 26, 2019 at 04:45:37PM +0100, swen wrote:
>On Tue, 2019-02-26 at 06:13 -0800, Jeremy Allison wrote:
>> On Tue, Feb 26, 2019 at 03:11:48PM +0100, Ralph Böhme wrote:
>> > If you're happy with the fixups, just fixup' em.
>> >
>> > 1. commit messages
>> > 2. doxygen comments
>> > 3. wait for Jeremy :)
>>
>> Being at the Vault Conference and working on SMB3+posix
>> bugs I've completely lost the plot on this one :-).
>>
>> Can you resend the full patchset when Ralph is
>> happy and I'll just re-review from scratch.
>
>The patchset contains now everything, I had to modify/fix and
>FIXUPs suggestd by Ralph B.
>
>I hope we have it all now.
>
>The patchset is compile tested and currently running at gitlab.
>
>Thanks to all to help out here !
>
>Please review and piush if happy.
thanks, much better!
There are still a few places where error is not initialized to 0. Can you check
that?
Furthermore, it seems in the commit
passdb: Use wrapper for string to integer conversion
the hunks changing source3/utils/net_registry.c and
source3/utils/net_rpc_registry.c should have been applied to the previous
commit.
Can you fix that any submit an updated patchset? Please also rerun CI with the
patchset and please include the link in the mail.
Finally, I'm not sure we want the last patch being part of this patchset. Once
strtoul_err() does that check, all callers must be updated to remove their own
checks for the same error.
Thanks for your patience with us nasty and grumpy reviewers! :)
-slow
--
Ralph Boehme, Samba Team https://samba.org/
Samba Developer, SerNet GmbH https://sernet.de/en/samba/
GPG-Fingerprint FAE2C6088A24252051C559E4AA1E9B7126399E46
More information about the samba-technical
mailing list