Bug #2757
closedrate_ctr_group_alloc with identical index doesn't return error
100%
Description
As seen by the bug fixed in Change-Id I1b56f8d7b47597ed263e6808074483edca0895de of osmo-bts.git, (https://gerrit.osmocom.org/#/c/5344) the rate_ctr_group_alloc() function appears to accept allocation of the same counter group for identical index, which is of course an error.
Let's make sure we return an error, and also make sure that all known callers actually deal with the error somehow (e.g. if the BTS counters couldn't be allocated, the entire BTS allocation should fail).
Updated by msuraev over 6 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 50
Fix for the library is available in gerrit 5418. We should also make sure that all the callers properly check and handle allocation error (which they should because there're other error paths in allocation function as well).
Updated by msuraev over 6 years ago
- Status changed from In Progress to Resolved
- % Done changed from 50 to 100
Checked that callers in osmo-msc, osmo-bsc, osmo-bts, osmo-sgsn, osmo-hlr, osmo-ggsn, osmo-pcu handle it properly.
Updated by msuraev over 6 years ago
- Status changed from Resolved to In Progress
- % Done changed from 100 to 80
OsmoBSC tests are failing after libosmocore upgrade which seems to be related.
Updated by msuraev over 6 years ago
Fix for osmo-bsc available in gerrit 5486. Test failure in osmo-pcu are next.
Updated by msuraev over 6 years ago
- % Done changed from 80 to 90
Gerrit 5486 has been merged, fix for osmo-pcu is available in gerrit 5487. The rest doesn't seem to be affected.
Updated by msuraev over 6 years ago
- Status changed from In Progress to Resolved
- % Done changed from 90 to 100
Updated by msuraev over 6 years ago
- Status changed from Resolved to In Progress
- % Done changed from 100 to 90
OsmoBSC fix got to be backported to OpenBSC.
Updated by msuraev about 6 years ago
- Status changed from Stalled to Resolved
- % Done changed from 90 to 100
Gerrit 5491 is merged.