linux-kernel - [PATCH 3/4] staging: csr: removed single line { } braces from for and if statements.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 17 Oct 2012 01:36:47 +0900
From:	Sangho Yi <antiroot@...il.com>
To:	devel@...verdev.osuosl.org
Cc:	gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org,
	Sangho Yi <antiroot@...il.com>
Subject: [PATCH 3/4] staging: csr: removed single line { } braces from for and if statements.

Yes, just removed the single line covering braces from the for() and
if(), and the corresponding else statements on mlme.c.

Signed-off-by: Sangho Yi <antiroot@...il.com>
---
 drivers/staging/csr/mlme.c |   20 +++++++-------------
 1 file changed, 7 insertions(+), 13 deletions(-)

diff --git a/drivers/staging/csr/mlme.c b/drivers/staging/csr/mlme.c
index dba12b2..e4ecdd8 100644
--- a/drivers/staging/csr/mlme.c
+++ b/drivers/staging/csr/mlme.c
@@ -89,11 +89,10 @@ unifi_mlme_wait_for_reply(unifi_priv_t *priv, ul_client_t *pcli,
 		} else {
 			/* Get the sequence number of the signal
 			 * that we previously set. */
-			if (pcli->seq_no != 0) {
+			if (pcli->seq_no != 0)
 				sent_seq_no = pcli->seq_no - 1;
-			} else {
+			else
 				sent_seq_no = 0x0F;
-			}
 
 			unifi_trace(priv, UDBG5,
 				"Received 0x%.4X, seq: (r:%d, s:%d)\n",
@@ -193,9 +192,8 @@ unifi_mlme_blocking_request(unifi_priv_t *priv, ul_client_t *pcli,
 	 * if the signal has been successfully set.
 	 */
 	pcli->seq_no++;
-	if (pcli->seq_no > 0x0F) {
+	if (pcli->seq_no > 0x0F)
 		pcli->seq_no = 0;
-	}
 
 	r = unifi_mlme_wait_for_reply(priv, pcli,
 		(sig->SignalPrimitiveHeader.SignalId + 1), timeout);
@@ -287,14 +285,12 @@ uf_abort_mlme(unifi_priv_t *priv)
 	priv->io_aborted = 1;
 
 	ul_cli = priv->netdev_client;
-	if (ul_cli) {
+	if (ul_cli)
 		wake_up_interruptible(&ul_cli->udi_wq);
-	}
 
 	ul_cli = priv->wext_client;
-	if (ul_cli) {
+	if (ul_cli)
 		wake_up_interruptible(&ul_cli->udi_wq);
-	}
 
 	return 0;
 } /* uf_abort_mlme() */
@@ -391,11 +387,9 @@ static const struct mlme_code Reason_codes[] = {
 static const char *
 lookup_something(const struct mlme_code *n, int id)
 {
-	for (; n->name; n++) {
-		if (n->id == id) {
+	for (; n->name; n++)
+		if (n->id == id)
 			return n->name;
-		}
-	}
 
 	/* not found */
 	return NULL;
-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/