Define in one and only one C file, declare in a header file.
authorGuy Harris <guy@alum.mit.edu>
Thu, 3 Mar 2016 01:37:02 +0000 (17:37 -0800)
committerGuy Harris <guy@alum.mit.edu>
Thu, 3 Mar 2016 01:38:10 +0000 (01:38 +0000)
Presumably the intent is to have the two dissectors share the tables in
question; if so, it's best done by defining the tables in one and only
one C file and declaring it in a header file included by both C files -
that 1) ensures the declaration and definition stay in sync and 2) keeps
the OS X build from failing.

Change-Id: Id2e7e5b7270c7109ffb091b2e16a631b83dde212
Reviewed-on: https://code.wireshark.org/review/14309
Reviewed-by: Guy Harris <guy@alum.mit.edu>
epan/dissectors/packet-gtp.h
epan/dissectors/packet-gtpv2.c

index 8b0af6783119d099afc9613044626f44a4277116..073afc9b0c094bc874456259a959ec2c1e33ebb9 100644 (file)
@@ -158,6 +158,12 @@ typedef struct session_args {
     guint8 last_cause;
 } session_args_t;
 
+/* Relation between frame -> session */
+extern GHashTable* session_table;
+
+/* Relation between <teid,ip> -> frame */
+extern wmem_tree_t* frame_tree;
+
 guint32 get_frame(address ip, guint32 teid, guint32 *frame);
 
 void remove_frame_info(guint32 *f);
index 9fb5efe09894b306879d89e3fad98441d16858f6..fecb9c5c351b459dcf4653d7a47b957d0272abd5 100644 (file)
@@ -1031,11 +1031,6 @@ static const value_string gtpv2_element_type_vals[] = {
 };
 static value_string_ext gtpv2_element_type_vals_ext = VALUE_STRING_EXT_INIT(gtpv2_element_type_vals);
 
-/* Relation between frame -> session */
-GHashTable* session_table;
-/* Relation between <teid,ip> -> frame */
-wmem_tree_t* frame_tree;
-
 typedef struct _gtpv2_hdr {
     guint8 flags;   /* GTP header flags */
     guint8 message; /* Message type */