Please use "error and out" logic (Re: [PATCH] Avoid privileged segfault in GetNCChanges and add many more tests)

[Andrew Bartlett]

On Thu, 2017-08-10 at 13:04 +0200, Stefan Metzmacher wrote:
> Hi Andrew,
> 
> here's my version with some comments in it.

Thanks,  I'll look at that next week (this 'simple fix' for the
bugzilla bug kind of ballooned!). 

> In general please check if we need the BUG: line
> in every commit, and move commits without towards the end of the
> patchset. 

The only commit needing the BUG is the one I tagged.  The rest is just
for master, to prove we don't regress in the future. 

> (The BUG: line should be before the Signed-off-by: line
> followed by an empty line...)

I don't mind some rules, but where did this one come from?

(I often add the BUG: tag with a script, which is why they end up
there). 

Thanks,

Andrew Bartlett
-- 
Andrew Bartlett                       http://samba.org/~abartlet/
Authentication Developer, Samba Team  http://samba.org
Samba Developer, Catalyst IT          http://catalyst.net.nz/services/samba