Improving performance of winbindd_list_groups
Gerald (Jerry) Carter
jerry at samba.org
Fri May 9 20:26:14 GMT 2008
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Volker Lendecke wrote:
> On Thu, Apr 24, 2008 at 05:51:13PM -0700, Steven Danneman wrote:
>> 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?
>
> Yep. Down below it calls vsnprintf somewhere, that
> terminates. I'd check in your patch, but I'd like Jerry to
> take a look and comment first.
Looks good to me (reading through) Have not tested if but
in theory I can follow the logic and it seems sound. I'm in
the ATL airport right now and don't really want to burn
battery booting VMs for testing. But I can deal with this
on Monday if you like. Or if Volker or someone else has
already tested it, I'm happy if they wanna check it into
the v3-3-test branch.
cheers, jerry
- --
=====================================================================
Samba ------- http://www.samba.org
Likewise Software --------- http://www.likewisesoftware.com
"What man is a man who does not make the world better?" --Balian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFIJLNmIR7qMdg1EfYRAjD5AKCJjzH9mCxeawobbObfG2uOj5VDEgCff6N2
TxLc59aMg/BhAp5YtAh+KCE=
=5hXJ
-----END PGP SIGNATURE-----
More information about the samba-technical
mailing list