Project

General

Profile

Actions

Bug #6014

closed

hnbgw_cnlink->state is never used

Added by neels about 1 year ago. Updated 6 months ago.

Status:
Resolved
Priority:
Low
Assignee:
Target version:
-
Start date:
04/25/2023
Due date:
% Done:

0%

Spec Reference:

Description

There is a hnbgw_cnlink->state member and a cnlink state machine concerned with sending RESET to RANAP/SCCP peers,
but the cnlink->state member is never set, this seems to be missing in hnbgw_cnlink_change_state().
We also never use its value, neither in state changes nor for vty_out().

It was like this ever since it was first created, see osmo-iuh.git 90256bad581a77069e0e9232927111f677a15272 and c4338deee9f3928eae61689687a1ff7ffee0d7b8 (in 2015, before migrating to a separate osmo-hnbgw.git)

Apparently all that is really needed is an event dispatch for CNLINK_S_EST_CONF, CNLINK_S_EST_RST_TX_WAIT_ACK and CNLINK_S_EST_ACTIVE
(see hnbgw_cnlink_change_state() in hnbgw_cn.c)

Figure out and verify that we can simply drop this parameter, and maybe replace the enum with simple direct function calls?

Actions #1

Updated by laforge 6 months ago

  • Assignee set to neels
Actions #2

Updated by neels 6 months ago

  • Status changed from New to Resolved

cnlink->state no longer exists since commit ade986280e52ce4514dcb16df9a8fd1ce63f929b "drop dead code: cnlink.T_RafC"

Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)