[sr-dev] git:master:a8be2459: ims_ipsec_pcscf: null terminate dst_uri fields

[Daniel-Constantin Mierla]

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

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2019-11-20T19:30:44+01:00

ims_ipsec_pcscf: null terminate dst_uri fields

---

Modified: src/modules/ims_ipsec_pcscf/cmd.c

---

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

---

diff --git a/src/modules/ims_ipsec_pcscf/cmd.c b/src/modules/ims_ipsec_pcscf/cmd.c
index 11673582ad..b72d57a0ad 100644
--- a/src/modules/ims_ipsec_pcscf/cmd.c
+++ b/src/modules/ims_ipsec_pcscf/cmd.c
@@ -753,13 +753,14 @@ int ipsec_forward(struct sip_msg* m, udomain_t* d)
 
     int buf_len = snprintf(buf, sizeof(buf) - 1, "sip:%.*s:%d", ci.via_host.len, ci.via_host.s, dst_port);
 
-    if((m->dst_uri.s = pkg_malloc(buf_len)) == NULL) {
+    if((m->dst_uri.s = pkg_malloc(buf_len + 1)) == NULL) {
         LM_ERR("Error allocating memory for dst_uri\n");
         goto cleanup;
     }
 
     memcpy(m->dst_uri.s, buf, buf_len);
     m->dst_uri.len = buf_len;
+    m->dst_uri.s[m->dst_uri.len] = '\0';
 
     // Set send socket
     struct socket_info * client_sock = grep_sock_info(via_host.af == AF_INET ? &ipsec_listen_addr : &ipsec_listen_addr6, src_port, dst_proto);