]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
reject: minor optimization of reset entry code
authorVictor Julien <victor@inliniac.net>
Fri, 12 Jun 2020 10:16:47 +0000 (12:16 +0200)
committerVictor Julien <victor@inliniac.net>
Sun, 28 Jun 2020 13:20:56 +0000 (15:20 +0200)
src/respond-reject.c

index 4d3308d6de6a2afec7c97042334faaa0a74b6f22..b3b37848b4260716791d8109672f491ad21e8189 100644 (file)
@@ -101,11 +101,8 @@ int RejectSendIPv4TCP(ThreadVars *tv, Packet *p, void *data)
         SCReturnInt(r);
     } else if(PACKET_TEST_ACTION(p, ACTION_REJECT_BOTH)) {
         int r = RejectSendLibnet11IPv4TCP(tv, p, data, REJECT_DIR_SRC);
-        if (RejectSendLibnet11IPv4TCP(tv, p, data, REJECT_DIR_DST) == 0) {
-            SCReturnInt(0);
-        } else {
-            SCReturnInt(r);
-        }
+        r |= RejectSendLibnet11IPv4TCP(tv, p, data, REJECT_DIR_DST);
+        SCReturnInt(r);
     }
     SCReturnInt(0);
 }
@@ -120,13 +117,9 @@ int RejectSendIPv4ICMP(ThreadVars *tv, Packet *p, void *data)
         int r = RejectSendLibnet11IPv4ICMP(tv, p, data, REJECT_DIR_DST);
         SCReturnInt(r);
     } else if(PACKET_TEST_ACTION(p, ACTION_REJECT_BOTH)) {
-        int r;
-        r = RejectSendLibnet11IPv4ICMP(tv, p, data, REJECT_DIR_SRC);
-        if (RejectSendLibnet11IPv4ICMP(tv, p, data, REJECT_DIR_DST) == 0) {
-            SCReturnInt(0);
-        } else {
-            SCReturnInt(r);
-        }
+        int r = RejectSendLibnet11IPv4ICMP(tv, p, data, REJECT_DIR_SRC);
+        r |= RejectSendLibnet11IPv4ICMP(tv, p, data, REJECT_DIR_DST);
+        SCReturnInt(r);
     }
     SCReturnInt(0);
 }
@@ -142,11 +135,8 @@ int RejectSendIPv6TCP(ThreadVars *tv, Packet *p, void *data)
         SCReturnInt(r);
     } else if(PACKET_TEST_ACTION(p, ACTION_REJECT_BOTH)) {
         int r = RejectSendLibnet11IPv6TCP(tv, p, data, REJECT_DIR_SRC);
-        if (RejectSendLibnet11IPv6TCP(tv, p, data, REJECT_DIR_DST) == 0) {
-            SCReturnInt(0);
-        } else {
-            SCReturnInt(r);
-        }
+        r |= RejectSendLibnet11IPv6TCP(tv, p, data, REJECT_DIR_DST);
+        SCReturnInt(r);
     }
     SCReturnInt(0);
 }
@@ -162,11 +152,8 @@ int RejectSendIPv6ICMP(ThreadVars *tv, Packet *p, void *data)
         SCReturnInt(r);
     } else if(PACKET_TEST_ACTION(p, ACTION_REJECT_BOTH)) {
         int r = RejectSendLibnet11IPv6ICMP(tv, p, data, REJECT_DIR_SRC);
-        if (RejectSendLibnet11IPv6ICMP(tv, p, data, REJECT_DIR_DST) == 0) {
-            SCReturnInt(0);
-        } else {
-            SCReturnInt(r);
-        }
+        r |= RejectSendLibnet11IPv6ICMP(tv, p, data, REJECT_DIR_DST);
+        SCReturnInt(r);
     }
     SCReturnInt(0);
 }