From ac56dfde20c5e689fb911debdd33a6155f7b130c Mon Sep 17 00:00:00 2001 From: Daniel-Constantin Mierla Date: Thu, 27 Aug 2020 12:34:50 +0200 Subject: [PATCH] usrloc: get_udomain() - safety checks for location table name parameter (cherry picked from commit 29dde9fea1eb21b4bb484286f7443dd32f611bd2) --- src/modules/usrloc/dlist.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/modules/usrloc/dlist.c b/src/modules/usrloc/dlist.c index cba8ec425c1..96a95fe3c97 100644 --- a/src/modules/usrloc/dlist.c +++ b/src/modules/usrloc/dlist.c @@ -792,13 +792,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; }