Author: rjung
Date: Tue Apr  1 16:10:55 2014
New Revision: 1583726

URL: http://svn.apache.org/r1583726
Log:
BZ 55696: Fix crash on Mac OS X 10.9 during config
parsing.

Modified:
    tomcat/jk/trunk/native/common/jk_map.c
    tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml

Modified: tomcat/jk/trunk/native/common/jk_map.c
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/native/common/jk_map.c?rev=1583726&r1=1583725&r2=1583726&view=diff
==============================================================================
--- tomcat/jk/trunk/native/common/jk_map.c (original)
+++ tomcat/jk/trunk/native/common/jk_map.c Tue Apr  1 16:10:55 2014
@@ -206,29 +206,27 @@ int jk_map_get_int(jk_map_t *m, const ch
     const char *rc;
     size_t len;
     int int_res;
-    int multit = 1;
 
     sprintf(buf, "%d", def);
     rc = jk_map_get_string(m, name, buf);
 
     len = strlen(rc);
     if (len) {
-        char *lastchar = &buf[0] + len - 1;
-        strcpy(buf, rc);
+        const char *lastchar = &rc[0] + len - 1;
+        int multit = 1;
         if ('m' == *lastchar || 'M' == *lastchar) {
-            *lastchar = '\0';
             multit = 1024 * 1024;
         }
         else if ('k' == *lastchar || 'K' == *lastchar) {
-            *lastchar = '\0';
             multit = 1024;
         }
-        int_res = atoi(buf);
+        /* Safe because atoi() will stop at any non-numeric lastchar */
+        int_res = atoi(rc) * multit;
     }
     else
         int_res = def;
 
-    return int_res * multit;
+    return int_res;
 }
 
 double jk_map_get_double(jk_map_t *m, const char *name, double def)

Modified: tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml?rev=1583726&r1=1583725&r2=1583726&view=diff
==============================================================================
--- tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml (original)
+++ tomcat/jk/trunk/xdocs/miscellaneous/changelog.xml Tue Apr  1 16:10:55 2014
@@ -57,6 +57,10 @@
       <fix>
         <bug>53542</bug>: ISAPI: Fix grammar in 503 error page. (rjung)
       </fix>
+      <fix>
+        <bug>55696</bug>: Crash on Mac OS X 10.9 during config parsing.
+        (rjung)
+      </fix>
     </changelog>
   </subsection>
 </section>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to