Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751502AbdIAHHh (ORCPT ); Fri, 1 Sep 2017 03:07:37 -0400 Received: from mail-io0-f195.google.com ([209.85.223.195]:37895 "EHLO mail-io0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751237AbdIAHHf (ORCPT ); Fri, 1 Sep 2017 03:07:35 -0400 X-Google-Smtp-Source: ADKCNb5VxV0thBCZS28Dy1XF8+/P2I+lVQBcu6cLkf0H7WPsr1X6fTUI1oww3zb7MaNyaw3077n6Ng== From: Janani Sankara Babu To: gregkh@linuxfoundation.org Cc: insafonov@gmail.com, gs051095@gmail.com, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org, Janani Sankara Babu Subject: [PATCH V2] staging:rtl8188eu:core Fix remove unneccessary else block Date: Fri, 1 Sep 2017 12:32:53 +0530 Message-Id: <1504249373-8143-1-git-send-email-jananis37@gmail.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1124 Lines: 34 This patch removes the unwanted braces and else statement inside the function 'SecIsInPMKIDList' Signed-off-by: Janani Sankara Babu --- Changelog V1 : Removed the else statement & Curly braces in If statement V2 : Removed the comment and added preincrement of i inside the while loop drivers/staging/rtl8188eu/core/rtw_mlme.c | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme.c b/drivers/staging/rtl8188eu/core/rtw_mlme.c index d3668ca..9c2571d 100644 --- a/drivers/staging/rtl8188eu/core/rtw_mlme.c +++ b/drivers/staging/rtl8188eu/core/rtw_mlme.c @@ -1708,14 +1708,9 @@ static int SecIsInPMKIDList(struct adapter *Adapter, u8 *bssid) do { if ((psecuritypriv->PMKIDList[i].bUsed) && - (!memcmp(psecuritypriv->PMKIDList[i].Bssid, bssid, ETH_ALEN))) { + (!memcmp(psecuritypriv->PMKIDList[i].Bssid, bssid, ETH_ALEN))) break; - } else { - i++; - /* continue; */ - } - - } while (i < NUM_PMKID_CACHE); + } while (++i < NUM_PMKID_CACHE); if (i == NUM_PMKID_CACHE) i = -1;/* Could not find. */ -- 1.9.1