[sr-dev] git:master:d51a3c43: ims_charging: fix wrong default User-Equipment-Info-Type AVP format

[Henning Westerholt]

Module: kamailio
Branch: master
Commit: d51a3c4393b345b783adccf8a0e960274e11afb2
URL: https://github.com/kamailio/kamailio/commit/d51a3c4393b345b783adccf8a0e960274e11afb2

Author: Henning Westerholt <hw at skalatan.de>
Committer: Henning Westerholt <hw at skalatan.de>
Date: 2020-07-11T11:13:03Z

ims_charging: fix wrong default User-Equipment-Info-Type AVP format

- fix User-Equipment-Info-Type AVP format
  According to the RFC 8506 it should be like this: The 48-bit Media Access
  Control (MAC) address is formatted as described in Section 3.21 of [RFC3580]
- use strlen instead of sizeof

(cherry picked from commit 1a8f54147063560f6016634152b17293394b8497)

---

Modified: src/modules/ims_charging/ims_ro.c

---

Diff:  https://github.com/kamailio/kamailio/commit/d51a3c4393b345b783adccf8a0e960274e11afb2.diff
Patch: https://github.com/kamailio/kamailio/commit/d51a3c4393b345b783adccf8a0e960274e11afb2.patch

---

diff --git a/src/modules/ims_charging/ims_ro.c b/src/modules/ims_charging/ims_ro.c
index 525e5ad5ef..05dc7f29c6 100644
--- a/src/modules/ims_charging/ims_ro.c
+++ b/src/modules/ims_charging/ims_ro.c
@@ -1615,9 +1615,8 @@ static int get_mac_avp_value(struct sip_msg *msg, str *value) {
 
     pv_parse_spec2(&mac_avp_name_str, &avp_spec, 1);
     if (pv_get_spec_value(msg, &avp_spec, &val) != 0 || val.rs.len == 0) {
-
-        value->s = "00:00:00:00:00:00";
-        value->len = sizeof ("00:00:00:00:00:00") - 1;
+        value->s = "00-00-00-00-00-00";
+        value->len = strlen(value->s);
         return -1;
     }