From bfd956ce6764c36687239cee3915fa0412c13a34 Mon Sep 17 00:00:00 2001 From: Shivani Bhardwaj Date: Fri, 11 Sep 2020 17:59:08 +0530 Subject: [PATCH] applayer/htp/xff: convert to FAIL/PASS API --- src/app-layer-htp-xff.c | 54 ++++++++++++++--------------------------- 1 file changed, 18 insertions(+), 36 deletions(-) diff --git a/src/app-layer-htp-xff.c b/src/app-layer-htp-xff.c index 73ddbe0363..e223af046e 100644 --- a/src/app-layer-htp-xff.c +++ b/src/app-layer-htp-xff.c @@ -260,90 +260,72 @@ static int XFFTest01(void) { char input[] = "1.2.3.4:5678"; char output[16]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "1.2.3.4") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "1.2.3.4") == 0); + PASS; } static int XFFTest02(void) { char input[] = "[12::34]:1234"; // thanks chort! char output[16]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "12::34") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "12::34") == 0); + PASS; } static int XFFTest03(void) { char input[] = "[2a03:2880:1010:3f02:face:b00c:0:2]:80"; // thanks chort! char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "2a03:2880:1010:3f02:face:b00c:0:2") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "2a03:2880:1010:3f02:face:b00c:0:2") == 0); + PASS; } static int XFFTest04(void) { char input[] = "[2a03:2880:1010:3f02:face:b00c:0:2]"; // thanks chort! char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "2a03:2880:1010:3f02:face:b00c:0:2") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "2a03:2880:1010:3f02:face:b00c:0:2") == 0); + PASS; } static int XFFTest05(void) { char input[] = "[::ffff:1.2.3.4]:1234"; // thanks double-p char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "::ffff:1.2.3.4") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "::ffff:1.2.3.4") == 0); + PASS; } static int XFFTest06(void) { char input[] = "12::34"; char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "12::34") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "12::34") == 0); + PASS; } static int XFFTest07(void) { char input[] = "1.2.3.4"; char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 1 && strcmp(output, "1.2.3.4") == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 1 && strcmp(output, "1.2.3.4") == 0); + PASS; } static int XFFTest08(void) { char input[] = "[1.2.3.4:1234"; char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 0); + PASS; } static int XFFTest09(void) { char input[] = "999.999.999.999:1234"; char output[46]; int r = ParseXFFString(input, output, sizeof(output)); - if (r == 0) { - return 1; - } - return 0; + FAIL_IF_NOT(r == 0); + PASS; } #endif