changed string_sub() to replace " ; and ` in the inserted string with _

use all_string_sub() if you don't want this.

diff --git a/source/include/proto.h b/source/include/proto.h
index 08476bdd8f708ed4fe0dfaa808c8ec5cf1052990..c4baa8f7a5082b59f58398f5eefde229f8bf1808 100644 (file)
--- a/source/include/proto.h
+++ b/source/include/proto.h
@@ -415,7 +415,8 @@ BOOL in_list(char *s,char *list,BOOL casesensitive);
 BOOL string_init(char **dest,const char *src);
 void string_free(char **s);
 BOOL string_set(char **dest,const char *src);
-BOOL string_sub(char *s,const char *pattern,const char *insert);
+void string_sub(char *s,const char *pattern,const char *insert);
+void all_string_sub(char *s,const char *pattern,const char *insert);
 void split_at_last_component(char *path, char *front, char sep, char *back);
 
 /*The following definitions come from  lib/util_unistr.c  */

diff --git a/source/lib/util_str.c b/source/lib/util_str.c
index c943a854cfa1900d0e73af543f864b6a4251d1be..dad0e854770f7d52298cbd353d187bf877660b16 100644 (file)
--- a/source/lib/util_str.c
+++ b/source/lib/util_str.c
@@ -990,6 +990,7 @@ BOOL string_set(char **dest,const char *src)
   return(string_init(dest,src));
 }
 
+
 /****************************************************************************
 substitute a string for a pattern in another string. Make sure there is 
 enough room!
@@ -997,31 +998,63 @@ enough room!
 This routine looks for pattern in s and replaces it with 
 insert. It may do multiple replacements.
 
-return True if a substitution was done.
+any of " ; or ` in the insert string are replaced with _
 ****************************************************************************/
-BOOL string_sub(char *s,const char *pattern,const char *insert)
+void string_sub(char *s,const char *pattern,const char *insert)
 {
-  BOOL ret = False;
-  char *p;
-  size_t ls,lp,li;
+       char *p;
+       size_t ls,lp,li, i;
+
+       if (!insert || !pattern || !s) return;
+
+       ls = strlen(s);
+       lp = strlen(pattern);
+       li = strlen(insert);
+
+       if (!*pattern) return;
+       
+       while (lp <= ls && (p = strstr(s,pattern))) {
+               memmove(p+li,p+lp,ls + 1 - (PTR_DIFF(p,s) + lp));
+               for (i=0;i<li;i++) {
+                       switch (insert[i]) {
+                       case '`':
+                       case '"':
+                       case ';':
+                               p[i] = '_';
+                               break;
+                       default:
+                               p[i] = insert[i];
+                       }
+               }
+               s = p + li;
+               ls += (li-lp);
+       }
+}
 
-  if (!insert || !pattern || !s) return(False);
 
-  ls = strlen(s);
-  lp = strlen(pattern);
-  li = strlen(insert);
+/****************************************************************************
+similar to string_sub() but allows for any character to be substituted. 
+Use with caution!
+****************************************************************************/
+void all_string_sub(char *s,const char *pattern,const char *insert)
+{
+       char *p;
+       size_t ls,lp,li, i;
 
-  if (!*pattern) return(False);
+       if (!insert || !pattern || !s) return;
 
-  while (lp <= ls && (p = strstr(s,pattern)))
-    {
-      ret = True;
-      memmove(p+li,p+lp,ls + 1 - (PTR_DIFF(p,s) + lp));
-      memcpy(p,insert,li);
-      s = p + li;
-      ls = strlen(s);
-    }
-  return(ret);
+       ls = strlen(s);
+       lp = strlen(pattern);
+       li = strlen(insert);
+
+       if (!*pattern) return;
+       
+       while (lp <= ls && (p = strstr(s,pattern))) {
+               memmove(p+li,p+lp,ls + 1 - (PTR_DIFF(p,s) + lp));
+               memcpy(p, insert, li);
+               s = p + li;
+               ls += (li-lp);
+       }
 }
 
 /****************************************************************************

diff --git a/source/smbd/chgpasswd.c b/source/smbd/chgpasswd.c
index 69ac69b59b174a1bd736f99cb2f82f2a27525fc8..30b9b3fed60aac3ddc2f67748295436ca502e7ae 100644 (file)
--- a/source/smbd/chgpasswd.c
+++ b/source/smbd/chgpasswd.c
@@ -434,12 +434,12 @@ BOOL chgpasswd(char *name,char *oldpass,char *newpass, BOOL as_root)
   }
 
   string_sub(passwordprogram,"%u",name);
-  string_sub(passwordprogram,"%o",oldpass);
-  string_sub(passwordprogram,"%n",newpass);
+  all_string_sub(passwordprogram,"%o",oldpass);
+  all_string_sub(passwordprogram,"%n",newpass);
 
   string_sub(chatsequence,"%u",name);
-  string_sub(chatsequence,"%o",oldpass);
-  string_sub(chatsequence,"%n",newpass);
+  all_string_sub(chatsequence,"%o",oldpass);
+  all_string_sub(chatsequence,"%n",newpass);
   return(chat_with_program(passwordprogram,name,chatsequence, as_root));
 }
 

diff --git a/source/web/statuspage.c b/source/web/statuspage.c
index 81564390a0c1c41051c261a75c1f84f907b7e869..faf1dcb20d613acf6a2baafecae5f1bd8611dc43 100644 (file)
--- a/source/web/statuspage.c
+++ b/source/web/statuspage.c
@@ -26,7 +26,7 @@ static char *tstring(time_t t)
 {
        static pstring buf;
        pstrcpy(buf, asctime(LocalTime(&t)));
-       string_sub(buf," "," ");
+       all_string_sub(buf," "," ");
        return buf;
 }