Add MPLS counter.

Check length before decoding each label.

diff --git a/src/decode-mpls.c b/src/decode-mpls.c
index f8f6993d7c3d34a42303e402957ff5d51ae12026..6e1da668f7a33d6da8b6db64506b22729fe9734c 100644 (file)
--- a/src/decode-mpls.c
+++ b/src/decode-mpls.c
@@ -34,19 +34,22 @@
 int DecodeMPLS(ThreadVars *tv, DecodeThreadVars *dtv, Packet *p, uint8_t *pkt,
     uint16_t len, PacketQueue *pq)
 {
-    if (len < MPLS_HEADER_LEN) {
-        return TM_ECODE_FAILED;
-    }
-
     uint32_t shim;
+    uint8_t ip_ver;
+
+    SCPerfCounterIncr(dtv->counter_mpls, tv->sc_perf_pca);
+
     do {
+        if (len < MPLS_HEADER_LEN) {
+            return TM_ECODE_FAILED;
+        }
         shim = *(uint32_t *)pkt;
         pkt += MPLS_HEADER_LEN;
         len -= MPLS_HEADER_LEN;
     } while (MPLS_BOTTOM(shim) == 0);
 
     /* Best guess at inner packet. */
-    uint8_t ip_ver = pkt[0] >> 4;
+    ip_ver = pkt[0] >> 4;
 
     switch (ip_ver) {
     case MPLS_PROTO_IPV4:

diff --git a/src/decode.c b/src/decode.c
index ae2041bc71bd304b6d4b139177bfd8c1f5310725..8ad78bcf32125222c5376379c4210757600d27e8 100644 (file)
--- a/src/decode.c
+++ b/src/decode.c
@@ -400,6 +400,8 @@ void DecodeRegisterPerfCounters(DecodeThreadVars *dtv, ThreadVars *tv)
                                                SC_PERF_TYPE_UINT64, "NULL");
     dtv->counter_ipv6inipv6 = SCPerfTVRegisterCounter("decoder.ipv6_in_ipv6", tv,
                                                SC_PERF_TYPE_UINT64, "NULL");
+    dtv->counter_mpls = SCPerfTVRegisterCounter("decoder.mpls", tv,
+                                                SC_PERF_TYPE_UINT64, "NULL");
     dtv->counter_avg_pkt_size = SCPerfTVRegisterAvgCounter("decoder.avg_pkt_size", tv,
                                                            SC_PERF_TYPE_UINT64, "NULL");
     dtv->counter_max_pkt_size = SCPerfTVRegisterMaxCounter("decoder.max_pkt_size", tv,

diff --git a/src/decode.h b/src/decode.h
index ce2f42d3149ed93a3b5cb4591349419810b1a8d6..98839be6bcdb67ac3338426e1ed1fcdf548259cb 100644 (file)
--- a/src/decode.h
+++ b/src/decode.h
@@ -597,6 +597,7 @@ typedef struct DecodeThreadVars_
     uint16_t counter_vlan_qinq;
     uint16_t counter_pppoe;
     uint16_t counter_teredo;
+    uint16_t counter_mpls;
     uint16_t counter_ipv4inipv6;
     uint16_t counter_ipv6inipv6;
     uint16_t counter_avg_pkt_size;