Bug 3782 - nmbd log file name changes on SIGHUP
Summary: nmbd log file name changes on SIGHUP
Status: NEW
Alias: None
Product: Samba 3.0
Classification: Unclassified
Component: nmbd (show other bugs)
Version: 3.0.1
Hardware: PPC AIX
: P3 minor
Target Milestone: none
Assignee: Samba Bugzilla Account
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-05-17 07:40 UTC by Heinrich Mislik
Modified: 2006-05-17 07:40 UTC (History)
0 users

See Also:

Attachments
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 Heinrich Mislik 2006-05-17 07:40:14 UTC 
In nmbd.c(main) there is
--------------
reopen_logs();

	DEBUG( 0, ( "Netbios nameserver version %s started.\n", SAMBA_VERSION_STRING) );
	DEBUGADD( 0, ( "%s\n", COPYRIGHT_STARTUP_MESSAGE ) );

	if ( !reload_nmbd_services(False) )
		return(-1);
---------------

Since reopen_logs is called befor reload_nmbd_services, the smb.conf file is not available and the logfile is opend where the -l option specifies.

After a SIGHUP reopen_logs is called again and the parameter "log file" in smb.conf is now used and logging goes into the same file as smbd. 

This is confusing. It should be decided and clearly documented, wether -l takes precedence over smb.conf or vice versa. Further a %-macro for the program name may be useful. log file = log.%x could then be used to separate log.smbd and log.nmbd