commit d7a083844db63b3dda412c755ce69bf13f41961f
author Stephen Hemminger <shemminger@vyatta.com> Wed Dec 07 00:09:56 2011 +0400
committer Denis Ovsienko <infrastation@yandex.ru> Wed Dec 07 01:24:09 2011 +0400
tree 1b3601cff5e6375f95d65b95f28afc0719fd6621
parent 04d5e24d7425a52f13abcb6d2a30aa6246d837e7

zebra: fix warnings in forward proc

Check result of fgets(), and don't set variable that is
never used.

zebra/ipforward_proc.c

diff --git a/zebra/ipforward_proc.c b/zebra/ipforward_proc.c
index 7384613..2876ede 100644
--- a/zebra/ipforward_proc.c
+++ b/zebra/ipforward_proc.c
@@ -45,7 +45,6 @@
 {
   FILE *fp;
   int ipforwarding = 0;
-  char *pnt;
   char buf[10];
 
   fp = fopen (proc_net_snmp, "r");
@@ -59,8 +58,8 @@
   /* 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 @@
   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;