]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
Minor optimizations to unified2 and fast.log.
authorVictor Julien <victor@inliniac.net>
Mon, 2 Apr 2012 13:45:16 +0000 (15:45 +0200)
committerVictor Julien <victor@inliniac.net>
Mon, 2 Apr 2012 13:45:16 +0000 (15:45 +0200)
src/alert-fastlog.c
src/alert-unified2-alert.c

index 256c16bba70f2f826688cb299ea507cb827ae0c6..c162482a972f82cd299271c3e7e7826d638a4824 100644 (file)
@@ -130,8 +130,11 @@ TmEcode AlertFastLogIPv4(ThreadVars *tv, Packet *p, void *data, PacketQueue *pq,
 
     CreateTimeString(&p->ts, timebuf, sizeof(timebuf));
 
-    SCMutexLock(&aft->file_ctx->fp_mutex);
+    char srcip[16], dstip[16];
+    PrintInet(AF_INET, (const void *)GET_IPV4_SRC_ADDR_PTR(p), srcip, sizeof(srcip));
+    PrintInet(AF_INET, (const void *)GET_IPV4_DST_ADDR_PTR(p), dstip, sizeof(dstip));
 
+    SCMutexLock(&aft->file_ctx->fp_mutex);
     aft->file_ctx->alerts += p->alerts.cnt;
 
     for (i = 0; i < p->alerts.cnt; i++) {
@@ -140,11 +143,6 @@ TmEcode AlertFastLogIPv4(ThreadVars *tv, Packet *p, void *data, PacketQueue *pq,
             continue;
         }
 
-        char srcip[16], dstip[16];
-
-        PrintInet(AF_INET, (const void *)GET_IPV4_SRC_ADDR_PTR(p), srcip, sizeof(srcip));
-        PrintInet(AF_INET, (const void *)GET_IPV4_DST_ADDR_PTR(p), dstip, sizeof(dstip));
-
         if (pa->action & ACTION_DROP && IS_ENGINE_MODE_IPS(engine_mode)) {
             action = "[Drop] ";
         } else if (pa->action & ACTION_DROP) {
index d07aaab9fde09bd187b17c8fc00f8f7242cd245d..7850a4b0fe751727364984a40d4249dba68098fe 100644 (file)
@@ -1064,13 +1064,10 @@ int Unified2IPv4TypeAlert (ThreadVars *tv, Packet *p, void *data, PacketQueue *p
             return -1;
         }
         fflush(aun->file_ctx->fp);
+        aun->file_ctx->alerts++;
         SCMutexUnlock(&aun->file_ctx->fp_mutex);
     }
 
-    SCMutexLock(&aun->file_ctx->fp_mutex);
-    aun->file_ctx->alerts += p->alerts.cnt;
-    SCMutexUnlock(&aun->file_ctx->fp_mutex);
-
     return 0;
 }