[PATCH] fix pointer error in appl_head loadparm.c
jra at dp.samba.org
jra at dp.samba.org
Fri Jan 17 06:38:01 GMT 2003
On Fri, Jan 17, 2003 at 05:12:28PM +1100, Martin Pool wrote:
> On 17 Jan 2003, Martin Pool <mbp at samba.org> wrote:
>
> > It seems that in the current code any caller to the lp_*_dos()
> > functions will get the wrong value!
>
> Correction: only the global lp_*dos parameters.
>
> Specfically:
>
> 1583:FN_GLOBAL_STRING_DOS(lp_serverstring_dos, &Globals.szServerString)
> 1599:FN_GLOBAL_CONST_STRING_DOS(lp_defaultservice_dos, &Globals.szDefaultService)
> 1604:FN_GLOBAL_CONST_STRING_DOS(lp_auto_services_dos, &Globals.szAutoServices)
> 1614:FN_GLOBAL_STRING_DOS(lp_logon_script_dos, &Globals.szLogonScript)
> 1616:FN_GLOBAL_STRING_DOS(lp_logon_path_dos, &Globals.szLogonPath)
> 1619:FN_GLOBAL_STRING_DOS(lp_logon_home_dos, &Globals.szLogonHome)
>
> So for example, on my PSA, smb.conf wants lp_serverstring_dos() to be
> set to "Hewlett Packard Print Server Appliance", but gdb shows that
> it's actually "". Presumably this is only a cosmetic problem.
>
> Similarly for logon_path and logon_home, though I suppose they don't
> matter for a print-only system.
Ok - looks good to me (sorry for the bug, it was my fault). If this
fixes the serverstring param please check it into APP-HEAD. It will
only be a problem in this branch.
Jeremy.
More information about the samba-technical
mailing list