From 280620d7ac30c411e05afe38307deef449720909 Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Tue, 18 Jan 2022 21:01:41 -0600 Subject: [PATCH] scan: Remove unreachable code We should never reach a point where NEW_SCAN_RESULTS or SCAN_ABORTED are received before a corresponding TRIGGER_SCAN is received. Even if this does happen, there's no harm from processing the commands anyway. --- src/scan.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/scan.c b/src/scan.c index 55dc2bc1..f45fa26c 100644 --- a/src/scan.c +++ b/src/scan.c @@ -1844,9 +1844,6 @@ static void scan_notify(struct l_genl_msg *msg, void *user_data) bool retry = false; bool get_results = false; - if (sc->state == SCAN_STATE_NOT_RUNNING) - break; - sc->state = SCAN_STATE_NOT_RUNNING; /* Was this our own scan or an external scan */ @@ -1936,9 +1933,6 @@ static void scan_notify(struct l_genl_msg *msg, void *user_data) break; case NL80211_CMD_SCAN_ABORTED: - if (sc->state == SCAN_STATE_NOT_RUNNING) - break; - sc->state = SCAN_STATE_NOT_RUNNING; if (sc->triggered) {