This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "The OpenGGSN project". The branch, neels/refactor has been updated discards fec3b14781797eb3d356b04566f473a94fec550f (commit) discards d7768c81ae99a0b7324562739aed0579537adc47 (commit) discards 746179370c89589ce9a4e959740a8f551a27d935 (commit) discards 5e0586df38fce1bb6edb734c1e4d1f4d9242a1f6 (commit) discards b8aae6dd3e0a0039953eb9c37f9bc720feca82e1 (commit) discards 7d62a4eaa315d689e3c1268f75c42d552f68ac81 (commit) discards c2058e0c648ff0c224da65bd1863da0b5dae542d (commit) via b72c2dc592bdaa2be86ea5b5f00a51599687c32e (commit) via b84545e30e327b429b7910dc4675b9e3e80df704 (commit) via ea458f4aa63e1a371a094d804ec4eb6ea6f8dfdd (commit) via 85c25ac91c5fe6e87e1c417ea3543149a92d4145 (commit) via 9a5cfa49ad1ce6d2cf6e9295a423e880d72dd1b5 (commit) via 878e188be73e49e96b2fe78604e7daa86cedf6ff (commit) via e845cb950544af8f0d00f5a2bed2072d795a2a7e (commit) via b18e811e24464d9b50f21674246806c3b7c25030 (commit) via 9b09738891770367f9878a1a4d1b83bc23217383 (commit) This update added new revisions after undoing existing revisions. That is to say, the old revision is not a strict subset of the new revision. This situation occurs when you --force push a change and generate a repository containing something like this: * -- * -- B -- O -- O -- O (fec3b14781797eb3d356b04566f473a94fec550f) \ N -- N -- N (b72c2dc592bdaa2be86ea5b5f00a51599687c32e) When this happens we assume that you've already had alert emails for all of the O revisions, and so we here report only the revisions in the N branch from the common base, B. Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://cgit.osmocom.org/openggsn/commit/?id=b72c2dc592bdaa2be86ea5b5f00a515… commit b72c2dc592bdaa2be86ea5b5f00a51599687c32e Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Mon Oct 19 13:26:39 2015 +0200 sgsnemu: improve an error message http://cgit.osmocom.org/openggsn/commit/?id=b84545e30e327b429b7910dc4675b9e… commit b84545e30e327b429b7910dc4675b9e3e80df704 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Mon Oct 19 13:24:51 2015 +0200 cosmetic: gtpie_gettv0(): flow readability http://cgit.osmocom.org/openggsn/commit/?id=ea458f4aa63e1a371a094d804ec4eb6… commit ea458f4aa63e1a371a094d804ec4eb6ea6f8dfdd Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Mon Oct 19 13:07:40 2015 +0200 gtpie_gettlv(): fix return value on specific error. Make gtpie_gettlv() return an error if gtpie_getie() returned an error. Previously, openggsn would fail to complain about certain missing elements. Technically, a missing IE could be detectable from the *length value, but the code in gtp.c relies on the return value to detect missing elements, which did not work prior to this commit. For example: if (gtpie_gettlv(ie, GTPIE_EUA, 0, &pdp->eua.l, &pdp->eua.v, sizeof(pdp->eua.v))) { gsn->missing++; GTP_LOGPKG(LOGL_ERROR, peer, pack, len, "Missing mandatory information field\n"); return gtp_create_pdp_resp(gsn, version, pdp, GTPCAUSE_MAN_IE_MISSING); } If an EUA were missing in this code path, openggsn would fail to issue an error message. Since pdp and hence pdp->eua.l is initialized as all-zero, it would probably not do much harm besides failing to issue an error. I haven't checked all callers though. http://cgit.osmocom.org/openggsn/commit/?id=85c25ac91c5fe6e87e1c417ea354314… commit 85c25ac91c5fe6e87e1c417ea3543149a92d4145 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Mon Oct 12 14:26:49 2015 +0200 comments: add/replace. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openggsn/commit/?id=9a5cfa49ad1ce6d2cf6e9295a423e88… commit 9a5cfa49ad1ce6d2cf6e9295a423e880d72dd1b5 Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Sun Oct 11 23:44:00 2015 +0200 comments: fix spelling/punctuation. Fix spelling/punctuation and one numbering in comments. Remove an opening brace from a comment to not mix up cindent in vim. Sponsored-by: On-Waves ehi http://cgit.osmocom.org/openggsn/commit/?id=878e188be73e49e96b2fe78604e7daa… commit 878e188be73e49e96b2fe78604e7daa86cedf6ff Author: Neels Hofmeyr <nhofmeyr(a)sysmocom.de> Date: Mon Oct 19 13:46:09 2015 +0200 rename gtp_dublicate (static func). Fix spelling dublicate -> duplicate in comments and in (apparently only statically used) gtp_dublicate(). Sponsored-by: On-Waves ehi ----------------------------------------------------------------------- Summary of changes: gtp/gtpie.c | 20 ++++++++++---------- sgsnemu/sgsnemu.c | 7 +++---- 2 files changed, 13 insertions(+), 14 deletions(-) hooks/post-receive -- The OpenGGSN project