[Fedora-directory-commits] adminserver/admserv/cgi-src40 sec-activate.c, 1.11, 1.12

[Richard Allen Megginson]

Author: rmeggins

Update of /cvs/dirsec/adminserver/admserv/cgi-src40
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv4922/adminserver/admserv/cgi-src40

Modified Files:
	sec-activate.c 
Log Message:
Resolves: bug 470946
Bug Description: admin console does not place quotes around the certificate name in console.conf
Reviewed by: nkinder (Thanks!)
Fix Description: Just put double quotes around the values we write out, including NSSNickname
Platforms tested: RHEL5
Flag Day: no
Doc impact: no



Index: sec-activate.c
===================================================================
RCS file: /cvs/dirsec/adminserver/admserv/cgi-src40/sec-activate.c,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -r1.11 -r1.12
--- sec-activate.c	14 Jul 2008 20:00:02 -0000	1.11
+++ sec-activate.c	4 Dec 2008 15:55:39 -0000	1.12
@@ -703,7 +703,7 @@
   while(fgets(inbuf, sizeof(inbuf), f) != NULL) {
     if (strncasecmp(inbuf,name,strlen(name)) == 0) { /* Line starts with the attribute name */
       if(val && *val != '\0') {
-        PR_snprintf(buf, sizeof(buf), "%s %s\n", name, val);
+        PR_snprintf(buf, sizeof(buf), "%s \"%s\"\n", name, val);
         lines[linecnt++] = strdup(buf);
         modified=1;
       }
@@ -718,7 +718,7 @@
   fclose(f);
 
   if (!modified && (val && *val != '\0')) { /* Add the attribute name/val pair*/
-    PR_snprintf(buf, sizeof(buf), "%s %s\n", name, val);
+    PR_snprintf(buf, sizeof(buf), "%s \"%s\"\n", name, val);
     lines[linecnt++] = strdup(buf);
   }