Attention is currently required from: fixeria, jolly, osmith, pespin.

pespin uploaded patch set #5 to this change.

View Change

The following approvals got outdated and were removed: Code-Review+1 by fixeria, Verified+1 by Jenkins Builder

ss7_asp_peer: Improvements in ss7_asp_peer_snprintf()

Convert it to osmo_strbuf, which simplifies the logic of the function,
taking osmo_multiaddr_ip_and_port_snprintf() as reference to implement it.
As a result:
* Single IPv6 addresses now get properly enclosed in [], eg:
  "[::]:2905".
* We avoid passing NULL to snprintf(), which is unspecified on some
  platforms.

Change-Id: I115951627adb0c6778a9a36159237c3eea130300
---
M src/ss7_asp_peer.c
M tests/ss7/ss7_test.err
M tests/ss7/ss7_test.ok
3 files changed, 31 insertions(+), 30 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/libosmo-sigtran refs/changes/53/43753/5

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