Attention is currently required from: pespin.

View Change

2 comments:

• File src/host/layer23/src/common/apn_fsm.c:

  • Patch Set #2, Line 103: apn_fsm_state_chg(fi, APN_ST_DISABLED);

    Don't we need to check *((bool *)data)? I guess the assumption is that it is false here, as I understand the BTS/other layers would send that to disable the APN, and then later on send it again with true to enable it again?

    Same for st_apn_active().

  • Patch Set #2, Line 139: apn_fsm_state_chg(fi, APN_ST_INACTIVE);

    • is this correct, receiving CTX_ACC and then going to inactive here?
    • what about APN_EV_RX_SM_ACT_PDP_CTX_REJ?

To view, visit change 32596. To unsubscribe, or for help writing mail filters, visit settings.