diff options
| author | paul <paul> | 2006-08-06 15:57:59 +0000 |
|---|---|---|
| committer | paul <paul> | 2006-08-06 15:57:59 +0000 |
| commit | 031b08d1f35f114f85b0697e3c3cfa2f6aa8472b (patch) | |
| tree | 19e138b9dd9d59c0de40c73381fe5aa65b4b39bd /zebra/if_proc.c | |
| parent | 2c5ad1b10b71dcf8606ca8cead31ee6e4246ac17 (diff) | |
| download | quagga-031b08d1f35f114f85b0697e3c3cfa2f6aa8472b.tar.bz2 quagga-031b08d1f35f114f85b0697e3c3cfa2f6aa8472b.tar.xz | |
[zebra] fix inconsistencies in ifstat_update_* declarations and definitions
2006-08-06 Paul Jakma <paul.jakma@sun.com>
* interface.h: (ifstat_update_proc) declaration should match
ifstat_update_sysctl really, which is to not return status, as
such status is not used anywhere.
* if_{proc,sysctl}.c: Make ifstat_update_* definitions and return values
consistent with each other and their declarations, ie:
(void) (*) (void).
Diffstat (limited to 'zebra/if_proc.c')
| -rw-r--r-- | zebra/if_proc.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/zebra/if_proc.c b/zebra/if_proc.c index 6002595a..367b0d6f 100644 --- a/zebra/if_proc.c +++ b/zebra/if_proc.c @@ -123,8 +123,8 @@ ifstat_dev_fields (int version, char *buf, struct interface *ifp) } /* Update interface's statistics. */ -int -ifstat_update_proc () +void +ifstat_update_proc (void) { FILE *fp; char buf[PROCBUFSIZ]; @@ -139,7 +139,7 @@ ifstat_update_proc () { zlog_warn ("Can't open proc file %s: %s", _PATH_PROC_NET_DEV, safe_strerror (errno)); - return -1; + return; } /* Drop header lines. */ @@ -162,7 +162,7 @@ ifstat_update_proc () ifstat_dev_fields (version, stat, ifp); } fclose(fp); - return 0; + return; } /* Interface structure allocation by proc filesystem. */ |