[pubcookie-dev] CVS update: jjminer; webiso/pubcookie/src index.cgi.c,1.187,1.188

[miner at doit.wisc.edu]

Update of /usr/local/cvsroot/webiso/pubcookie/src
 In directory webiso-cvs.cac.washington.edu:/var/tmp/cvs-serv2134
 
 Modified Files:
 	index.cgi.c 
 Log Message:
     Brad discovered that we aren't allocating enough space for the NULL at the
     end of the altconfig..  Somehow never manifested itself until we moved to
     the new servers.
 



Index: webiso/pubcookie/src/index.cgi.c
diff -u webiso/pubcookie/src/index.cgi.c:1.187 webiso/pubcookie/src/index.cgi.c:1.188
--- webiso/pubcookie/src/index.cgi.c:1.187	Thu May 22 09:29:33 2008
+++ webiso/pubcookie/src/index.cgi.c	Fri Jan 15 09:17:03 2010
@@ -18,7 +18,7 @@
 /** @file index.cgi.c
  * Login server CGI
  *
- * $Id: index.cgi.c,v 1.187 2008/05/22 16:29:33 fox Exp $
+ * $Id: index.cgi.c,v 1.188 2010/01/15 17:17:03 jjminer Exp $
  */
 
 #ifdef WITH_FCGI
@@ -2156,8 +2156,8 @@
     /* I don't remember if config_init will change altconfig, better safe than
      * sorry -- tmp_config is a pointer to the static environment */
     if (tmp_config != NULL) {
-        altconfig = pbc_malloc (p, strlen (tmp_config));
-        strncpy (altconfig, tmp_config, strlen (tmp_config));
+        altconfig = pbc_malloc (p, strlen (tmp_config) + 1);
+        strncpy (altconfig, tmp_config, strlen (tmp_config) + 1);
     }
 
     libpbc_config_init (p, altconfig, "logincgi");



end of message