]> git.ipfire.org Git - people/ms/suricata.git/commitdiff
alert: silence compiler type warning
authorSascha Steinbiss <sascha@steinbiss.name>
Wed, 8 Feb 2017 11:40:00 +0000 (12:40 +0100)
committerVictor Julien <victor@inliniac.net>
Wed, 8 Feb 2017 22:15:50 +0000 (23:15 +0100)
The `ts_ecr' and `ts_val' struct fields are integer types, not
pointers. This leads GCC 6.3.0 to complain about comparisons to
NULL.

Signed-off-by: Sascha Steinbiss <sascha@steinbiss.name>
src/alert-prelude.c

index 2526320ac344661c0a83b4272fe26da853cdd06a..9c55b2815417acf1e2ff3fcaff1baeab14030f5b 100644 (file)
@@ -512,10 +512,10 @@ static int PacketToData(const Packet *p, const PacketAlert *pa, idmef_alert_t *a
             AddIntData(alert, "tcp_win", TCP_GET_WINDOW(p));
             AddIntData(alert, "tcp_sum", TCP_GET_SUM(p));
             AddIntData(alert, "tcp_urp", TCP_GET_URG_POINTER(p));
-            if (p->tcpvars.ts_val != NULL) {
+            if (p->tcpvars.ts_val != 0) {
                 AddIntData(alert, "tcp_tsval", TCP_GET_TSVAL(p));
             }
-            if (p->tcpvars.ts_ecr != NULL) {
+            if (p->tcpvars.ts_ecr != 0) {
                 AddIntData(alert, "tcp_tsecr", TCP_GET_TSECR(p));
             }
             if (p->tcph != NULL) {
@@ -961,4 +961,3 @@ void AlertPreludeRegister (void)
         AlertPreludeThreadInit, AlertPreludeThreadDeinit, NULL);
 }
 #endif /* PRELUDE */
-