Attention is currently required from: laforge, pespin.

pespin uploaded patch set #3 to this change.

View Change

The following approvals got outdated and were removed: Verified-1 by Jenkins Builder

Fix combined linkset lookup when inserting route

The early loop termiantion logic was wrong, which meant
ss7_route_table_find_combined_linkset() may return earlier than expected
and hence a new combined linkset may have been created when adding a
route instead of adding it to a previous existing matching one.

The fix can be spotted by the fact that now "show cs7 instance 0 route"
shows the 3 3.2.0/11 prio=1 routes properly in the same combined linkset
in osmo_stp_route_prio.vty.

Change-Id: I44d7545a3c6a3ea4839e983104112f8afa6b981a
---
M src/ss7_route_table.c
M tests/vty/osmo_stp_route_prio.vty
2 files changed, 12 insertions(+), 5 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/51/40551/3

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

Gerrit-MessageType: newpatchset
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I44d7545a3c6a3ea4839e983104112f8afa6b981a
Gerrit-Change-Number: 40551
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Attention: laforge <laforge@osmocom.org>
Gerrit-Attention: pespin <pespin@sysmocom.de>