osmo-ggsn.git branch pespin/ippool created. 1.0.0-42-g503f468

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, pespin/ippool has been created at 503f468366fc28720208aeee957bfb32a50a60d8 (commit) - Log ----------------------------------------------------------------- http://cgit.osmocom.org/osmo-ggsn/commit/?id=503f468366fc28720208aeee957bfb… commit 503f468366fc28720208aeee957bfb32a50a60d8 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri Oct 13 14:32:09 2017 +0200 ippool: Implement and use blacklist instead of blindly using IPPOOL_NOGATEWAY Commit dda21ed7d4a897c9284c69175d0da598598eae40 modified previous calls to ippool_new() removing the pass of flags to avoid allocating certain problematic IPs from the pool to MS, such as the network, gateway and broadcast IPs. Today I did some unsucessful tests with osmo-ggsn with a pool "ip prefix dynamic 176.16.222.0/24", and thus IP 176.16.222.0 was being assigned to the MS. De-capsulated DNS packets were received in the tun interface, but the Linux system in there was unable to correctly forward the packets to the gateway interface connected to the Internet. However, adding a second MS which got 176.16.222.1 had its packets forwarded correctly. However, previous implementation relies on flag IPPOOL_NOGATEWAY flag to blindly blacklist first IP after the network ip (ie, .0 and .1 are removed), which limits the IP reserved for the tun device to be .1. If a different IP in the range is assigned, it may cause issues. As a result, a blacklist is introduced in this commit to dynamically fetch the tun IP address and exlucde it from the pool of available IPs. Change-Id: I8e91f7280d60490c858a769dd578c1c8e54e9243 http://cgit.osmocom.org/osmo-ggsn/commit/?id=d6ede37b7e62ca7e3544b7319b9234… commit d6ede37b7e62ca7e3544b7319b9234c564365dca Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri Oct 13 11:56:16 2017 +0200 lib/ippool: Fix listsize calculated 1 elem too small Take the chance this commit is changing test output to also remove use of IPPOOL_NOGATEWAY which is going to be removed soon, and instead test IPPOOL_NOBROADCAST. Change-Id: I95c24bc690490155bec9e3933d678e4668d7745f http://cgit.osmocom.org/osmo-ggsn/commit/?id=81ff18c9d7dde7541963922ddd22de… commit 81ff18c9d7dde7541963922ddd22de4bba2eca58 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri Oct 13 14:32:24 2017 +0200 Remove trailing whitespace Change-Id: I8e24f95a88bef3a59006a89c219871e6156963d7 http://cgit.osmocom.org/osmo-ggsn/commit/?id=6f7a46a9b603dbf01fce3b68247756… commit 6f7a46a9b603dbf01fce3b6824775601e0cc2c70 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri Oct 13 11:18:59 2017 +0200 tests: Makefile.am: Add missing osmocom includedir Change-Id: Ibc2f98fe06366c94342831205fc741675dc2dda6 http://cgit.osmocom.org/osmo-ggsn/commit/?id=4a76d4bd5dd73eaafcfcdc30bef170… commit 4a76d4bd5dd73eaafcfcdc30bef1703e35383977 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Thu Oct 12 16:42:46 2017 +0200 lib/tun.c: tun_ipv6_linklocal_get(): fix memory leak with getifaddrs() From getifaddrs(3) man: "The data returned by getifaddrs() is dynamically allocated and should be freed using freeifaddrs() when no longer needed" Change-Id: If6300d1c8d36fcafef294a4c11bbda31a158bb9c http://cgit.osmocom.org/osmo-ggsn/commit/?id=1d4a2f8cf40f33425a9ea364a27ecc… commit 1d4a2f8cf40f33425a9ea364a27ecc00528a0337 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri Oct 13 13:09:38 2017 +0200 tests: in46a_test: Add missing expected output file Change-Id: I542f7a201df169b703519e800b1aab8deb95ea77 http://cgit.osmocom.org/osmo-ggsn/commit/?id=5c891eb0e778f35233c55da065c31d… commit 5c891eb0e778f35233c55da065c31d3fc76d0db9 Author: Pau Espin Pedrol <pespin(a)sysmocom.de> Date: Fri Oct 13 13:10:15 2017 +0200 tests: Remove Makefile.in Change-Id: Ie21e8bfab7cda52aa696dd26280efc05c4b51bba http://cgit.osmocom.org/osmo-ggsn/commit/?id=adabfb2f4e6b7e50376c9fd3ca47d4… commit adabfb2f4e6b7e50376c9fd3ca47d4d4b9bd8337 Author: Harald Welte <laforge(a)gnumonks.org> Date: Thu Oct 12 20:01:54 2017 +0800 add Unit test for lib/in46_addr.c code Change-Id: Id032c330405e5dca8ecfc0970d128341ed75c675 http://cgit.osmocom.org/osmo-ggsn/commit/?id=336972cb622caf96aa7e1cbf35e5fb… commit 336972cb622caf96aa7e1cbf35e5fbcc58b19285 Author: Harald Welte <laforge(a)gnumonks.org> Date: Thu Oct 12 19:37:14 2017 +0800 update .gitignore Change-Id: I030bd616dd27d49e55b5bdcc7c0b4fa3eb523da4 http://cgit.osmocom.org/osmo-ggsn/commit/?id=1eb209c09b24b88c26deea4baec543… commit 1eb209c09b24b88c26deea4baec543ab68928426 Author: Harald Welte <laforge(a)gnumonks.org> Date: Thu Oct 12 19:30:49 2017 +0800 Add ippool unit-test for 'make check' runs This test creates a variety of v4 (and one v6) pool and performs allocations until the pool is full, then frees a random number of randomly distributed addresses, re-allocates them and again checks that they're all available and finally once the pool is full allocations fail. Change-Id: Ibf4588e8f3ae71684e5262c0caaa2689aee73a94 ----------------------------------------------------------------------- hooks/post-receive -- The OpenGGSN project