zebra: fix warnings in forward proc

Check result of fgets(), and don't set variable that is
never used.
This commit is contained in:
Stephen Hemminger 2011-12-07 00:09:56 +04:00 committed by Denis Ovsienko
parent 8502229b7e
commit 7e7a2d38b2

View 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;