]> git.ipfire.org Git - thirdparty/xtables-addons.git/commitdiff
xt_ACCOUNT: remove redundant braces
authorJan Engelhardt <jengelh@inai.de>
Mon, 9 Nov 2015 21:32:32 +0000 (22:32 +0100)
committerJan Engelhardt <jengelh@inai.de>
Mon, 9 Nov 2015 21:33:49 +0000 (22:33 +0100)
For single-line statements, the {} are not strictly needed.

extensions/ACCOUNT/xt_ACCOUNT.c

index e4678fc0cb0c66caa3af8acd3a43f23d214daff4..5c3d9436b2a0b51efe8d38c02d0babf6d6938a35 100644 (file)
@@ -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);
                        }
                }