pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: 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.

Change-Id: I338a700e705f15bec962dca5ea69533ddec5e8a6
---
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 39618. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I338a700e705f15bec962dca5ea69533ddec5e8a6
Gerrit-Change-Number: 39618
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>