7022269: clean up fscanf usage in Linux networking native code
authorchegar
Mon, 28 Feb 2011 11:03:52 +0000
changeset 8554 14e626e9407b
parent 8553 46c2babb1e44
child 8555 389ce5f9a6d1
7022269: clean up fscanf usage in Linux networking native code Reviewed-by: michaelm, alanb
jdk/src/solaris/native/java/net/NetworkInterface.c
jdk/src/solaris/native/java/net/net_util_md.c
--- a/jdk/src/solaris/native/java/net/NetworkInterface.c	Fri Feb 25 02:06:10 2011 -0800
+++ b/jdk/src/solaris/native/java/net/NetworkInterface.c	Mon Feb 28 11:03:52 2011 +0000
@@ -1099,7 +1099,7 @@
 #ifdef AF_INET6
 static netif *enumIPv6Interfaces(JNIEnv *env, int sock, netif *ifs) {
     FILE *f;
-    char addr6[40], devname[20];
+    char addr6[40], devname[21];
     char addr6p[8][5];
     int plen, scope, dad_status, if_idx;
     uint8_t ipv6addr[16];
--- a/jdk/src/solaris/native/java/net/net_util_md.c	Fri Feb 25 02:06:10 2011 -0800
+++ b/jdk/src/solaris/native/java/net/net_util_md.c	Mon Feb 28 11:03:52 2011 +0000
@@ -608,7 +608,7 @@
     {
         /* now find the scope_id for "lo" */
 
-        char devname[20];
+        char devname[21];
         char addr6p[8][5];
         int plen, scope, dad_status, if_idx;
 
@@ -651,7 +651,7 @@
 static void initLocalIfs () {
     FILE *f;
     unsigned char staddr [16];
-    char ifname [32];
+    char ifname [33];
     struct localinterface *lif=0;
     int index, x1, x2, x3;
     unsigned int u0,u1,u2,u3,u4,u5,u6,u7,u8,u9,ua,ub,uc,ud,ue,uf;
@@ -660,7 +660,7 @@
         return ;
     }
     while (fscanf (f, "%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x%2x "
-                "%d %x %x %x %s",&u0,&u1,&u2,&u3,&u4,&u5,&u6,&u7,
+                "%d %x %x %x %32s",&u0,&u1,&u2,&u3,&u4,&u5,&u6,&u7,
                 &u8,&u9,&ua,&ub,&uc,&ud,&ue,&uf,
                 &index, &x1, &x2, &x3, ifname) == 21) {
         staddr[0] = (unsigned char)u0;
@@ -1102,7 +1102,7 @@
      * index.
      */
     if (match) {
-        char devname[20];
+        char devname[21];
         char addr6p[8][5];
         int plen, scope, dad_status, if_idx;