[PATCH] Add stackframes to public libsmbclient functions

Jeremy Allison jra at samba.org
Wed Nov 28 06:55:44 GMT 2007


On Wed, Nov 28, 2007 at 07:50:27AM +0100, Volker Lendecke wrote:
> 
> That's indeed very clever, thanks. If I had seen this
> proposal when I first added talloc_stackframe() I would
> probably have removed the stackframe thing immediately.

I'm very glad you didn't :-).

> On the other hand, talloc_tos() at least to me has proved to
> be very handy and in particular very robust. We've had quite
> a few memleaks due to temporary talloc contexts not being
> freed in obscure error returns. This does not mean that I
> intend to be sloppy, but it's a cozy feeling that things
> can't go really bad.

Indeed - it is a very handy way to convert older code
to talloc contexts - I'm finding it essential to keep
the diff size down whilst removing pstrings.

> I think Jeremy and Jerry should also comment here I think :-)

Just did :-).

Jeremy.


More information about the samba-technical mailing list