This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
Pau Espin Pedrol gerrit-no-reply at lists.osmocom.orgHello Jenkins Builder, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/6200 to look at the new patch set (#2). ggsn.c: cb_tun_ind: Convert ifelse to switch statement Change-Id: I4fc97f367ae615cdf0a9501f89419990c2fd4599 --- M ggsn/ggsn.c 1 file changed, 6 insertions(+), 3 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-ggsn refs/changes/00/6200/2 diff --git a/ggsn/ggsn.c b/ggsn/ggsn.c index de8090b..c7f6376 100644 --- a/ggsn/ggsn.c +++ b/ggsn/ggsn.c @@ -726,13 +726,15 @@ struct ip6_hdr *ip6h = (struct ip6_hdr *)pack; struct ippool_t *pool; - if (iph->version == 4) { + switch (iph->version) { + case 4: if (len < sizeof(*iph) || len < 4*iph->ihl) return -1; dst.len = 4; dst.v4.s_addr = iph->daddr; pool = apn->v4.pool; - } else if (iph->version == 6) { + break; + case 6: /* Due to the fact that 3GPP requires an allocation of a * /64 prefix to each MS, we must instruct * ippool_getip() below to match only the leading /64 @@ -740,7 +742,8 @@ dst.len = 8; dst.v6 = ip6h->ip6_dst; pool = apn->v6.pool; - } else { + break; + default: LOGP(DTUN, LOGL_NOTICE, "non-IPv%u packet received from tun\n", iph->version); return -1; } -- To view, visit https://gerrit.osmocom.org/6200 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I4fc97f367ae615cdf0a9501f89419990c2fd4599 Gerrit-PatchSet: 2 Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de> Gerrit-Reviewer: Jenkins Builder