-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Jeremy Allison wrote:
| On Thu, Apr 24, 2008 at 05:51:13PM -0700, Steven Danneman wrote:
|>> Looks very good to me, thanks!
|>> Just a quick comment: In listgroups_recv you're using talloc_asprintf
|> to paste the
|>> results together. These strings can get *very* long, so you might want
|> to use
|>> talloc_asprintf_append_buffer, which does not use strlen for the
|> existing string.
|>
|>> Jerry, what do you think about this patch?
|>> Volker
|> Thanks Volker,
|>
|> That's a good suggestion for even better performance. Attached is an
|> updated patch using talloc_asprintf_append_buffer(). I'm wasn't
|> positive from the API description that the returned string will be NULL
|> terminated, but in all my tests it seems to be. Is this a correct
|> assumption?
|
| Jerry, just waiting on your go ahead to push this one...

I know. Thanks for waiting.



jerry

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (Darwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFIF3rtIR7qMdg1EfYRAhj9AKCWcZK9mILJZ16W9Ar9FP MxvXvRKwCgzV8f
YXpjEd4GkjR7gBP5+EKEDek=
=CcfH
-----END PGP SIGNATURE-----