It makes no sense to have a % substiution in the 'smb ports' parameter.
Andrew Bartlett
/* The following definitions come from param/loadparm.c */
-char *lp_smb_ports(void);
+const char *lp_smb_ports(void);
const char *lp_dos_charset(void);
const char *lp_unix_charset(void);
const char *lp_display_charset(void);
#define FN_LOCAL_CHAR(fn_name,val) \
char fn_name(const struct share_params *p) {return(LP_SNUM_OK(p->service)? ServicePtrs[(p->service)]->val : sDefault.val);}
-FN_GLOBAL_STRING(lp_smb_ports, &Globals.smb_ports)
+FN_GLOBAL_CONST_STRING(lp_smb_ports, &Globals.smb_ports)
FN_GLOBAL_CONST_STRING(lp_dos_charset, &Globals.dos_charset)
FN_GLOBAL_CONST_STRING(lp_unix_charset, &Globals.unix_charset)
FN_GLOBAL_CONST_STRING(lp_display_charset, &Globals.display_charset)