Project

General

Profile

Actions

Bug #3787

open

Assignment Request modifies conn->codec_list before the Assignment is successful

Added by neels about 5 years ago. Updated about 4 years ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
-
Target version:
-
Start date:
02/07/2019
Due date:
% Done:

0%

Spec Reference:

Description

In osmo-bsc, all transient state around Assignment should live in conn->assignment.*
and only become definitive valid conn->* items upon a successful Assignment;
if Assignment fails, conn->* should remain unaffected.

In current master, I notice that this item does not heed this separation properly:

bssmap_handle_assignm_req(): rc = gsm0808_dec_speech_codec_list(&conn->codec_list, ...)
The conn's codec list is modified before the Assignment was successful. Instead, there should be a conn->assignment.codec_list.
The only other place using conn->codec_list is handover_decision_2.c: it attempts to find out what codec types are supported.
Maybe it doesn't make sense to look up merely the last Assignment's received codec_list at all?
Anyway, if it does, then we should still not overwrite conn->codec_list until an Assignment was successful, for sanity.


Related issues

Related to OsmoBSC - Bug #3833: Fix storage location of AMR S15-S0 bitsResolvedneels03/11/2019

Actions
Actions #1

Updated by laforge almost 5 years ago

  • Assignee set to dexter
Actions #2

Updated by laforge about 4 years ago

  • Assignee deleted (dexter)
Actions #3

Updated by neels about 4 years ago

  • Related to Bug #3833: Fix storage location of AMR S15-S0 bits added
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)