Project

General

Profile

Actions

Bug #4172

open

osmoggsn crashes when stopping

Added by lynxis over 4 years ago. Updated over 4 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
08/23/2019
Due date:
% Done:

0%

Spec Reference:

Description

on the cccamp 2019 the ggsn crashed.
Direct before that, the sgsn crashed.

(gdb) bt
#0  gtp_delete_context_req2 (gsn=0x0, pdp=0x7f9e66094138, cbp=0x0, teardown=1)
    at ../../../src/osmo-ggsn/gtp/gtp.c:2449
#1  0x000055702696abf6 in pool_close_all_pdp (pool=0x557027b1ed90) at ../../../src/osmo-ggsn/ggsn/ggsn.c:111
#2  0x000055702696cc17 in apn_stop (apn=0x557027b1eaa0) at ../../../src/osmo-ggsn/ggsn/ggsn.c:124
#3  0x000055702696ce60 in ggsn_stop (ggsn=0x557027b1e650) at ../../../src/osmo-ggsn/ggsn/ggsn.c:1182
#4  0x0000557026968a16 in ggsn_stop (ggsn=0x557027b1e650) at ../../../src/osmo-ggsn/ggsn/ggsn.c:1177
#5  main (argc=3, argv=0x7fffd20cd068) at ../../../src/osmo-ggsn/ggsn/ggsn.c:1319
Actions #1

Updated by pespin over 4 years ago

  • Subject changed from osmoggsn crashs when stopping to osmoggsn crashes when stopping
Actions #2

Updated by pespin over 4 years ago

This one looks really weird. pdp->gsn should never be NULL, since pdp is always related to a gsn_t. In no place pdp->gsn is set to NULL, and actually only place where pdp->gsn is set is in gtp_pdp_newpdp():

(*pdp)->gsn = gsn;

And gsn cannot be NULL there because it's deferred to get access to pdpa before in the same function:

struct pdp_t *pdpa = gsn->pdpa;

However, pdp_freepdp() does this:

memset(pdp, 0, sizeof(struct pdp_t));

So only way pdp->gsn is NULL is that pdp_freepdp was already called on that pdp context.

osmo-ggsn only makes use of pdp_freepdp() in one place and it's not triggered here: cb_conf(type=GTP_DELETE_PDP_REQ), basically because we free the pdp context before that point whenever we do a DeletePdpReq in osmo-ggsn. For other cases, osmo-ggsn calls gtp_freepdp_teardown() which would call the delete_ctx cb and appropiatelly remove it from ippool.

libgtp never calls pdp_freepdp() (which does not call delete_ctx cb), and it should only do it when the user/app doesn't have knowledge abot that pdp context (so no associated structures are managed).

Would be great to know which commit/version was this one running when the crash appeared.

Some log file would have been great too, it would be a lot more clear to understand what happened.

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)