diff options
author | Stephen Hemminger <shemminger@vyatta.com> | 2011-12-07 00:09:56 +0400 |
---|---|---|
committer | Denis Ovsienko <infrastation@yandex.ru> | 2011-12-07 01:24:09 +0400 |
commit | d7a083844db63b3dda412c755ce69bf13f41961f (patch) | |
tree | 1b3601cff5e6375f95d65b95f28afc0719fd6621 /zebra | |
parent | 04d5e24d7425a52f13abcb6d2a30aa6246d837e7 (diff) |
zebra: fix warnings in forward proc
Check result of fgets(), and don't set variable that is
never used.
Diffstat (limited to 'zebra')
-rw-r--r-- | zebra/ipforward_proc.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/zebra/ipforward_proc.c b/zebra/ipforward_proc.c index 73846137..2876eded 100644 --- a/zebra/ipforward_proc.c +++ b/zebra/ipforward_proc.c @@ -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; |