Change in ...osmo-ggsn[master]: gtp: Add missing headers

laforge gerrit-no-reply at lists.osmocom.org
Thu Jun 20 16:48:13 UTC 2019


laforge has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/14556 )

Change subject: gtp: Add missing headers
......................................................................

gtp: Add missing headers

Those headers are using types defined in other places (like sockaddr_in)
and don't explicitly include them, which makes future queue_test fail.

Change-Id: I65e12a067d89ef71be3719636b64f4d93ea73cc4
---
M gtp/pdp.h
M gtp/queue.h
2 files changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved



diff --git a/gtp/pdp.h b/gtp/pdp.h
index 9527464..d64d394 100644
--- a/gtp/pdp.h
+++ b/gtp/pdp.h
@@ -14,6 +14,7 @@
 #define _PDP_H
 
 #include <stdbool.h>
+#include <netinet/in.h>
 
 #include <osmocom/core/defs.h>
 
diff --git a/gtp/queue.h b/gtp/queue.h
index ec5f3de..76cb7be 100644
--- a/gtp/queue.h
+++ b/gtp/queue.h
@@ -17,6 +17,8 @@
 #ifndef _QUEUE_H
 #define _QUEUE_H
 
+#include "gtp.h"
+
 #define QUEUE_DEBUG 0		/* Print debug information */
 
 #define QUEUE_SIZE 1024		/* Size of retransmission queue */

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/14556
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: I65e12a067d89ef71be3719636b64f4d93ea73cc4
Gerrit-Change-Number: 14556
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190620/554fa89a/attachment.html>


More information about the gerrit-log mailing list