Project

General

Profile

Actions

Bug #3786

open

Assignment Request modifies conn->lcls state 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 call does not heed this separation properly:

bssmap_handle_assignm_req(): bssmap_handle_ass_req_lcls(conn, &tp);
This apparently changes the LCLS state before the Assignment is accepted as a valid request.
It seems to me that

  • bssmap_handle_ass_req_lcls() part evaluating the IEs should at first store the IE information in conn->assignment.* instead of in conn->*, and should copy to conn->lcls in assignment_success();
  • and that lcls_update_config() should be called in assignment_success() near the end.
Actions #1

Updated by laforge almost 5 years ago

  • Assignee set to dexter
  • Priority changed from Normal to Low
Actions #2

Updated by laforge about 4 years ago

  • Assignee deleted (dexter)
Actions

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 48.8 MB)