Bug 4836 - [PATCH] Wrong output in error message on nss_info parsing
Summary: [PATCH] Wrong output in error message on nss_info parsing
Status: RESOLVED FIXED
Alias: None
Product: Samba 3.0
Classification: Unclassified
Component: winbind (show other bugs)
Version: 3.0.25a
Hardware: All All
: P3 trivial
Target Milestone: none
Assignee: Samba Bugzilla Account
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
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
View All Add an attachment (proposed patch, testcase, etc.)

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