pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, approved
snm: Fix generating wrong format in 'affected PCs' log string

The previous code wrongly added an extra ", " string between each
element.

(cherry picked from commit b8a27a9b09babdf219e76347e89b66a9214f7b08)
Change-Id: I339e7a94ffe56a37ad0bebc1b8067023d33a882b
---
M src/xua_snm.c
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/xua_snm.c b/src/xua_snm.c
index 36fe3c7..971380b 100644
--- a/src/xua_snm.c
+++ b/src/xua_snm.c
@@ -59,7 +59,7 @@
uint8_t mask = _aff_pc >> 24;

/* append point code + mask */
- out = talloc_asprintf_append(out, "%s%s/%u, ", i == 0 ? "" : ", ",
+ out = talloc_asprintf_append(out, "%s%s/%u", i == 0 ? "" : ", ",
osmo_ss7_pointcode_print(s7i, pc), mask);
}
return out;

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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: pespin/rel-2.1.0
Gerrit-Change-Id: I339e7a94ffe56a37ad0bebc1b8067023d33a882b
Gerrit-Change-Number: 40399
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: pespin <pespin@sysmocom.de>