Module: kamailio
Branch: 5.0
Commit: 0ff6b710d0a5729778b3de1a15666e29d73aec56
URL: 
https://github.com/kamailio/kamailio/commit/0ff6b710d0a5729778b3de1a15666e29d73aec56

Author: Daniel-Constantin Mierla <mico...@gmail.com>
Committer: Daniel-Constantin Mierla <mico...@gmail.com>
Date: 2017-03-21T02:43:06+01:00

textops: safety check of allocated memory inside append_time_request_f()

- reported by GH #1040

(cherry picked from commit a51a32633429d761f8bc742386b6f6bcb6b38522)

---

Modified: src/modules/textops/textops.c

---

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

---

diff --git a/src/modules/textops/textops.c b/src/modules/textops/textops.c
index 9338fac..1ebc20d 100644
--- a/src/modules/textops/textops.c
+++ b/src/modules/textops/textops.c
@@ -1214,6 +1214,10 @@ static int append_time_request_f(struct sip_msg* msg, 
char* p1, char *p2)
        }
 
        time_str.s = pkg_malloc(MAX_TIME);
+       if(time_str.s==NULL) {
+               LM_ERR("no more pkg memory\n");
+               goto error;
+       }
        time_str.len=strftime(time_str.s, MAX_TIME, TIME_FORMAT, bd_time);
        if (time_str.len>MAX_TIME-2 || time_str.len==0) {
                LM_ERR("unexpected time length\n");


_______________________________________________
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