]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
eve/json: always output vlan field as array
authorMark Janssen <mark@praseodym.net>
Sun, 10 Feb 2019 14:29:42 +0000 (15:29 +0100)
committerVictor Julien <victor@inliniac.net>
Tue, 23 Apr 2019 07:55:17 +0000 (09:55 +0200)
src/output-json.c

index a91d6eb9824ec1e0db70f97974ae05c1a9b6302f..cf0fafd5cb0f9d5d98f0bfa8aa100c8c9231300c 100644 (file)
@@ -716,27 +716,13 @@ json_t *CreateJSONHeader(const Packet *p, enum OutputJsonLogDirection dir,
     }
 
     /* vlan */
-    if (p->vlan_idx > 0) {
-        json_t *js_vlan;
-        switch (p->vlan_idx) {
-            case 1:
-                json_object_set_new(js, "vlan",
-                                    json_integer(VLAN_GET_ID1(p)));
-                break;
-            case 2:
-                js_vlan = json_array();
-                if (unlikely(js != NULL)) {
-                    json_array_append_new(js_vlan,
-                                    json_integer(VLAN_GET_ID1(p)));
-                    json_array_append_new(js_vlan,
-                                    json_integer(VLAN_GET_ID2(p)));
-                    json_object_set_new(js, "vlan", js_vlan);
-                }
-                break;
-            default:
-                /* shouldn't get here */
-                break;
+    if (f->vlan_idx > 0) {
+        json_t *js_vlan = json_array();
+        json_array_append_new(js_vlan, json_integer(f->vlan_id[0]));
+        if (f->vlan_idx > 1) {
+            json_array_append_new(js_vlan, json_integer(f->vlan_id[1]));
         }
+        json_object_set_new(js, "vlan", js_vlan);
     }
 
     /* 5-tuple */