Bug 4836 - [PATCH] Wrong output in error message on nss_info parsing
[PATCH] Wrong output in error message on nss_info parsing
Status: RESOLVED FIXED
Product: Samba 3.0
Classification: Unclassified
Component: winbind
3.0.25a
All All
: P3 trivial
: none
Assigned To: Samba Bugzilla Account
Samba QA Contact
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-07-30 15:59 UTC by Matthijs Kooijman
Modified: 2007-07-31 00:12 UTC (History)
0 users

See Also:


Attachments
Trivial fixing patch (305 bytes, patch)
2007-07-30 16:00 UTC, Matthijs Kooijman
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matthijs Kooijman 2007-07-30 15:59:30 UTC
in nsswitch/nss_info.h:156 I find:

	for ( i=0; nss_list && nss_list[i]; i++ ) {

					if ( !parse_nss_parm(nss_list[i], &backend, &domain) )  {
							DEBUG(0,("nss_init: failed to parse \"%s\"!\n",
									 nss_list[0]));
							continue;
					}
	[...]

This loops all elements of nss_list, tries to parse the current element
(nss_list[i]) and if it fails, gives an error about the first element
(nss_list[0]). This should obviously be the current, not the first element.

See attached (trivial) patch that fixes this.
Comment 1 Matthijs Kooijman 2007-07-30 16:00:08 UTC
Created attachment 2847 [details]
Trivial fixing patch
Comment 2 Volker Lendecke 2007-07-31 00:12:14 UTC
Checked in with r24087.

Thanks,

Volker