[sr-dev] git:master:e9c2022b: Merge pull request #338 from doublec/htable_overrun

Daniel-Constantin Mierla miconda at gmail.com
Wed Sep 23 14:42:01 CEST 2015


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

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2015-09-23T14:41:56+02:00

Merge pull request #338 from doublec/htable_overrun

htable: Fix buffer overrun in htable_rpc_list

---

Modified: modules/htable/htable.c

---

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

---

diff --git a/modules/htable/htable.c b/modules/htable/htable.c
index c6e2250..8339e41 100644
--- a/modules/htable/htable.c
+++ b/modules/htable/htable.c
@@ -1044,7 +1044,7 @@ static void  htable_rpc_list(rpc_t* rpc, void* c)
 		if (ht->dbtable.len > 0) {
 			len = ht->dbtable.len > 127 ? 127 : ht->dbtable.len;
 			memcpy(dbname, ht->dbtable.s, len);
-			dbname[ht->dbtable.len] = '\0';
+			dbname[len] = '\0';
 		} else {
 			dbname[0] = '\0';
 		}




More information about the sr-dev mailing list