From 819e37237ac2b6e2fd76f1dba8a4cb97866b1f34 Mon Sep 17 00:00:00 2001 From: lesl Date: Thu, 7 Jan 2021 09:49:04 +0800 Subject: [PATCH] wifi: Fix removeIfaceInstanceFromBridgedApIfaceInternal Fix the incorrect empty check. Bug: 162686273 Test: Manual Test, shutdown instance works normally Change-Id: I44c5c4adcc7a964e25b5fa5e291bb9b9f4932655 --- wifi/1.5/default/wifi_chip.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/wifi/1.5/default/wifi_chip.cpp b/wifi/1.5/default/wifi_chip.cpp index 6dd400c899..a37fee22a4 100644 --- a/wifi/1.5/default/wifi_chip.cpp +++ b/wifi/1.5/default/wifi_chip.cpp @@ -1031,7 +1031,7 @@ WifiStatus WifiChip::removeIfaceInstanceFromBridgedApIfaceInternal( const std::string& ifname, const std::string& ifInstanceName) { legacy_hal::wifi_error legacy_status; const auto iface = findUsingName(ap_ifaces_, ifname); - if (!iface.get() || !ifInstanceName.empty()) { + if (!iface.get() || ifInstanceName.empty()) { return createWifiStatus(WifiStatusCode::ERROR_INVALID_ARGS); } // Requires to remove one of the instance in bridge mode