Review at

sgsnemu: Fix bad ptr during context deallocation

Older commit switched pdp_t to have an array of 2 peers instead of
only one in order to accomodate for ipv4v6 contexts, which can have 2
addresses assigned. The usage of peer field was not updated in sgsnemu
accordingly, which means the wrong memory portion was being accessed.

Fixes: 2d6a69e69a4b4cb2b8cc63c4810dae44e5a4d8f6 ("Add support for IPv4v6 End 
User Addresses")

Change-Id: I9e563522173a82b265e93b1ef9dc93ced40fefa2
M sgsnemu/sgsnemu.c
1 file changed, 2 insertions(+), 2 deletions(-)

  git pull ssh:// refs/changes/92/6392/1

diff --git a/sgsnemu/sgsnemu.c b/sgsnemu/sgsnemu.c
index cf4aa44..804b542 100644
--- a/sgsnemu/sgsnemu.c
+++ b/sgsnemu/sgsnemu.c
@@ -1312,8 +1312,8 @@
        if (tun && options.ipdown)
                tun_runscript(tun, options.ipdown);
-       ipdel((struct iphash_t *)pdp->peer);
-       memset(pdp->peer, 0, sizeof(struct iphash_t));  /* To be sure */
+       ipdel((struct iphash_t *)pdp->peer[0]);
+       memset(pdp->peer[0], 0, sizeof(struct iphash_t));       /* To be sure */
        if (1 == options.contexts)
                state = 5;      /* Disconnected */

To view, visit
To unsubscribe, visit

Gerrit-MessageType: newchange
Gerrit-Change-Id: I9e563522173a82b265e93b1ef9dc93ced40fefa2
Gerrit-PatchSet: 1
Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <>

Reply via email to