For hysterical raisins you must use string_set() to set the value of a

string in the loadparam Globals struct.  Using pstrcpy was causing every
NULL string was being set to the name of the winbindd log file.  (-:
(This used to be commit 24bae9f05523a7c85bf1988d349149ebeb5067f0)

diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c
index c900e99af41b84f371edb76635791b5d27364473..0f32787b126397d8b96f9fb8378a51e20f073a93 100644 (file)
--- a/source3/param/loadparm.c
+++ b/source3/param/loadparm.c
@@ -3889,7 +3889,7 @@ BOOL is_netbios_alias_or_name(const char *name)
 void lp_set_logfile(const char *name)
 {
        extern pstring debugf;
-       pstrcpy(Globals.szLogFile, name);
+       string_set(&Globals.szLogFile, name);
        pstrcpy(debugf, name);
 }