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

Author: Daniel-Constantin Mierla <mico...@gmail.com>
Committer: Daniel-Constantin Mierla <mico...@gmail.com>
Date: 2016-12-01T12:37:06+01:00

ss7ops: safety check before freeing allocated isup json buffer

---

Modified: modules/ss7ops/ss7ops_mod.c

---

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

---

diff --git a/modules/ss7ops/ss7ops_mod.c b/modules/ss7ops/ss7ops_mod.c
index 26aaeb6..efeb816 100644
--- a/modules/ss7ops/ss7ops_mod.c
+++ b/modules/ss7ops/ss7ops_mod.c
@@ -61,7 +61,7 @@ struct mtp_level_3_hdr {
  * ss7 module - helper module to convert M2UA/ISUP to JSON
  *
  */
-static const char *isup_last;
+static const char *isup_last = NULL;
 
 static int w_isup_to_json(struct sip_msg* _m, char* param1, char* param2);
 static int pv_get_isup(struct sip_msg *msg, pv_param_t *param, pv_value_t 
*res);
@@ -259,7 +259,7 @@ static int w_isup_to_json(struct sip_msg *_m, char *param1, 
char *param2)
        int opc, dpc, mtp_type, int_len, rc;
        size_t len;
 
-       free((char *) isup_last);
+       if(isup_last) free((char *) isup_last);
        isup_last = NULL;
 
        data = fetch_payload(_m, "$var(payload)", &int_len);


_______________________________________________
sr-dev mailing list
sr-dev@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to