From 99e21004eacebe3cf57cd5f7c06e58d33c9d06bd Mon Sep 17 00:00:00 2001
From: "Nicholas J. Kain" <nicholas@kain.us>
Date: Sat, 10 May 2014 21:13:24 -0400
Subject: [PATCH] arp_min_close_fd() will always force the arp fd to be equal
to -1, so there is no need to check force_reopen twice.
---
src/arp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/arp.c b/src/arp.c
index 41525bd..f32b36c 100644
--- a/src/arp.c
+++ b/src/arp.c
@@ -220,7 +220,7 @@ static void arp_switch_state(struct client_state_t *cs, arp_state_t state)
bool force_reopen = state == AS_DEFENSE || garp.state == AS_DEFENSE;
if (force_reopen)
arp_min_close_fd(cs);
- if (cs->arpFd < 0 || force_reopen) {
+ if (cs->arpFd < 0) {
if (arp_open_fd(cs, state) < 0)
suicide("%s: (%s) Failed to open arpFd when changing state %u -> %u",
client_config.interface, __func__, garp.state, state);