[PATCH] CTDB - typo and cid fixes (bug 13554)
Volker.Lendecke at SerNet.DE
Thu Aug 2 06:43:33 UTC 2018
On Thu, Aug 02, 2018 at 04:38:03PM +1000, Amitay Isaacs wrote:
> On Thu, Aug 2, 2018 at 4:29 PM, Volker Lendecke
> <Volker.Lendecke at sernet.de> wrote:
> > On Thu, Aug 02, 2018 at 03:20:52PM +1000, Amitay Isaacs via samba-technical wrote:
> >> Hi,
> >> Please review and push.
> > Shouldn't the second one add a "goto done" to avoid short-term leaks?
> > Likewise for the other "return ENOMEM" checks?
> The cmdline_run() function creates a temporary context for the cmdline
> functions and then frees the temorary context. So there is no need to
> do anything special in each individual cmdline function.
Why all the talloc_free's after done: then, and there *is* a "goto
done:" inside instead of a "return ret;". This is at least not 100%
Apart from that bike-shedding, RB+.
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de
More information about the samba-technical