Browse Source

usrloc: get_udomain() - safety checks for location table name parameter

(cherry picked from commit 29dde9fea1eb21b4bb484286f7443dd32f611bd2)
Daniel-Constantin Mierla 5 years ago
parent
commit
3ce17b3ba0
1 changed files with 11 additions and 0 deletions
  1. 11 0
      src/modules/usrloc/dlist.c

+ 11 - 0
src/modules/usrloc/dlist.c

@@ -597,13 +597,24 @@ int get_udomain(const char* _n, udomain_t** _d)
 	dlist_t* d;
 	str s;
 
+	if(_n == NULL) {
+		LM_ERR("null location table name\n");
+		goto notfound;
+	}
+
 	s.s = (char*)_n;
 	s.len = strlen(_n);
+	if(s.len <= 0) {
+		LM_ERR("empty location table name\n");
+		goto notfound;
+	}
 
 	if (find_dlist(&s, &d) == 0) {
 		*_d = d->d;
 		return 0;
 	}
+
+notfound:
 	*_d = NULL;
 	return -1;
 }