[PATCH] Make loadparm more common
obnox at samba.org
Wed Apr 2 04:46:15 MDT 2014
On 2014-04-02 at 11:43 +1300, Andrew Bartlett wrote:
> On Tue, 2014-04-01 at 08:43 +0200, Volker Lendecke wrote:
> > This has a talloc_zero(NULL,...). Would it be possible to
> > use talloc_tos() here?
> No, I'm not willing to add any more talloc_tos() to this area of the
> code. Almost all the odd unexpected failures caused by this patch set
> were due to new talloc_tos() calls, because not all callers had a
> Additionally, it would have to be un-done again before the code is made
> common later in the series, and we don't use talloc_tos() in the top
> level code.
I don't quite get that: talloc_tos() is from lib/ and several
top-level subsystems already use it, why couldn't this code use
it as well?
I'd say, for a samba-interal top-level lib/subsystem, we should
use the talloc stack. But I may be missing an important point.
Cheers - Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 215 bytes
Desc: Digital signature
More information about the samba-technical