diff options
author | Felix Fietkau <nbd@openwrt.org> | 2014-03-21 15:54:59 +0000 |
---|---|---|
committer | Felix Fietkau <nbd@openwrt.org> | 2014-03-21 15:54:59 +0000 |
commit | 5a6f00b84669e8a095aee650c022a473a81f6069 (patch) | |
tree | 4ec8e0295a2861fe9c4f002dd4b30db7719a3a71 /package/network/services/ead/src/ead.c | |
parent | 7c029cc26290ee490fae85bbc413c3b0849927eb (diff) | |
download | upstream-5a6f00b84669e8a095aee650c022a473a81f6069.tar.gz upstream-5a6f00b84669e8a095aee650c022a473a81f6069.tar.bz2 upstream-5a6f00b84669e8a095aee650c022a473a81f6069.zip |
ead: clean up
Remove unused variables
Correct some (non important) memory leaks
Remove fclose(NULL)
Correct possible out of bound access
spotted with cppcheck
Signed-off-by: Etienne CHAMPETIER <etienne.champetier@free.fr>
SVN-Revision: 39994
Diffstat (limited to 'package/network/services/ead/src/ead.c')
-rw-r--r-- | package/network/services/ead/src/ead.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/package/network/services/ead/src/ead.c b/package/network/services/ead/src/ead.c index 36235207bc..d1847c1b5e 100644 --- a/package/network/services/ead/src/ead.c +++ b/package/network/services/ead/src/ead.c @@ -120,8 +120,8 @@ set_recv_type(pcap_t *p, bool rx) #ifdef PACKET_RECV_TYPE struct sockaddr_ll sll; struct ifreq ifr; - int ifindex, mask; - int fd, ret; + int mask; + int fd; fd = pcap_get_selectable_fd(p); if (fd < 0) @@ -132,7 +132,7 @@ set_recv_type(pcap_t *p, bool rx) else mask = 0; - ret = setsockopt(fd, SOL_PACKET, PACKET_RECV_TYPE, &mask, sizeof(mask)); + setsockopt(fd, SOL_PACKET, PACKET_RECV_TYPE, &mask, sizeof(mask)); #endif } @@ -841,7 +841,7 @@ static int check_bridge_port(const char *br, const char *port, void *arg) { struct ead_instance *in; - struct list_head *p, *tmp; + struct list_head *p; list_for_each(p, &instances) { in = list_entry(p, struct ead_instance, list); @@ -873,7 +873,7 @@ check_all_interfaces(void) { #ifdef linux struct ead_instance *in; - struct list_head *p, *tmp; + struct list_head *p; br_foreach_bridge(check_bridge, NULL); |