From 1dc2a1c2de9b33401ec6850d74ff49f0f3a4e675 Mon Sep 17 00:00:00 2001 From: Jan Engelhardt Date: Mon, 9 Nov 2015 22:32:32 +0100 Subject: [PATCH] xt_ACCOUNT: remove redundant braces For single-line statements, the {} are not strictly needed. --- extensions/ACCOUNT/xt_ACCOUNT.c | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/extensions/ACCOUNT/xt_ACCOUNT.c b/extensions/ACCOUNT/xt_ACCOUNT.c index e4678fc..5c3d943 100644 --- a/extensions/ACCOUNT/xt_ACCOUNT.c +++ b/extensions/ACCOUNT/xt_ACCOUNT.c @@ -115,10 +115,8 @@ static void *ipt_acc_zalloc_page(void) // Don't use get_zeroed_page until it's fixed in the kernel. // get_zeroed_page(GFP_ATOMIC) void *mem = (void *)__get_free_pages(GFP_ATOMIC, 2); - if (mem) { + if (mem != NULL) memset(mem, 0, 2 *PAGE_SIZE); - } - return mem; } @@ -139,11 +137,9 @@ static void ipt_acc_data_free(void *data, uint8_t depth) if (depth == 1) { struct ipt_acc_mask_16 *mask_16 = data; unsigned int b; - for (b = 0; b <= 255; b++) { - if (mask_16->mask_24[b]) { + for (b = 0; b <= 255; ++b) + if (mask_16->mask_24[b]) free_pages((unsigned long)mask_16->mask_24[b], 2); - } - } free_pages((unsigned long)data, 2); return; } @@ -156,11 +152,9 @@ static void ipt_acc_data_free(void *data, uint8_t depth) struct ipt_acc_mask_16 *mask_16 = ((struct ipt_acc_mask_8 *)data)->mask_16[a]; - for (b = 0; b <= 255; b++) { - if (mask_16->mask_24[b]) { + for (b = 0; b <= 255; ++b) + if (mask_16->mask_24[b]) free_pages((unsigned long)mask_16->mask_24[b], 2); - } - } free_pages((unsigned long)mask_16, 2); } }