reversed test in nss_init
Herb Lewis
hlewis at panasas.com
Wed Sep 10 16:34:25 GMT 2008
You are right. The "try again" comment confused me.
Gerald (Jerry) Carter wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Herb,
>
> This is incorrect. The check I believe is to skip plugins that
> fail to init and continue processing the remaining ones in the
> list.
>
>
>
> cheers, jerry
>
> Herb Lewis wrote:
>
>>$ git-diff --unified=10
>>diff --git a/source/winbindd/nss_info.c b/source/winbindd/nss_info.c
>>index daa3dd0..96a2c77 100644
>>--- a/source/winbindd/nss_info.c
>>+++ b/source/winbindd/nss_info.c
>>@@ -156,21 +156,21 @@ static bool parse_nss_parm( const char *config,
>>char **backend, char **domain )
>> DEBUG(0,("nss_init: failed to parse \"%s\"!\n",
>> nss_list[i]));
>> continue;
>> }
>>
>> /* validate the backend */
>>
>> if ( (nss_backend = nss_get_backend( backend )) == NULL ) {
>> /* attempt to register the backend */
>> status = smb_probe_module( "nss_info", backend );
>>- if ( !NT_STATUS_IS_OK(status) ) {
>>+ if ( NT_STATUS_IS_OK(status) ) {
>> continue;
>> }
>>
>> /* try again */
>> if ( (nss_backend = nss_get_backend( backend ))
>>== NULL ) {
>> DEBUG(0,("nss_init: unregistered backend
>>%s!. Skipping\n",
>> backend));
>> continue;
>> }
>>
>
>
>
> - --
> =====================================================================
> 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
>
> iD8DBQFIx+NmIR7qMdg1EfYRAnEbAJ9qA+uEv8OqYHSIi00HJU47s2GYCQCfee9X
> JEBi5TRLvZuESjqRXKMqo3Q=
> =NlFP
> -----END PGP SIGNATURE-----
More information about the samba-technical
mailing list