]> git.proxmox.com Git - mirror_frr.git/commitdiff
zebra: fix warnings in forward proc
authorStephen Hemminger <shemminger@vyatta.com>
Tue, 6 Dec 2011 20:09:56 +0000 (00:09 +0400)
committerDenis Ovsienko <infrastation@yandex.ru>
Tue, 6 Dec 2011 21:24:09 +0000 (01:24 +0400)
Check result of fgets(), and don't set variable that is
never used.

zebra/ipforward_proc.c

index 73846137a8b8188f5a9dd48e46dbb70af100530a..2876eded378048b95039d2b93a8e47a0ca1abaac 100644 (file)
@@ -45,7 +45,6 @@ ipforward (void)
 {
   FILE *fp;
   int ipforwarding = 0;
-  char *pnt;
   char buf[10];
 
   fp = fopen (proc_net_snmp, "r");
@@ -59,8 +58,8 @@ ipforward (void)
   /* Get ip_statistics.IpForwarding : 
      1 => ip forwarding enabled 
      2 => ip forwarding off. */
-  pnt = fgets (buf, 6, fp);
-  sscanf (buf, "Ip: %d", &ipforwarding);
+  if (fgets (buf, 6, fp))
+    sscanf (buf, "Ip: %d", &ipforwarding);
 
   fclose(fp);
   
@@ -140,8 +139,8 @@ ipforward_ipv6 (void)
   if (fp == NULL)
     return -1;
 
-  fgets (buf, 2, fp);
-  sscanf (buf, "%d", &ipforwarding);
+  if (fgets (buf, 2, fp))
+    sscanf (buf, "%d", &ipforwarding);
 
   fclose (fp);
   return ipforwarding;