diff options
| author | Sean Bright <sean.bright@gmail.com> | 2017-09-01 10:51:06 -0400 |
|---|---|---|
| committer | Sean Bright <sean.bright@gmail.com> | 2017-09-01 11:16:09 -0400 |
| commit | 5f4863d4f9b707ba66bf94302f641d3aee438f7c (patch) | |
| tree | 792af82e4816ed1a107e5fdfdbeab71f6dce1c17 /apps/app_directory.c | |
| parent | 07577fc9972da2d91d9cd8d16611705dd6c1253c (diff) | |
app_directory: Handle a NULL mailbox without crashing
ASTERISK-27241 #close
Reported by: David Moore
Change-Id: Ibbbca85517b04c315406ebfe3b6f7e0763daedc6
Diffstat (limited to 'apps/app_directory.c')
| -rw-r--r-- | apps/app_directory.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/apps/app_directory.c b/apps/app_directory.c index b75e85763..9cfcbf862 100644 --- a/apps/app_directory.c +++ b/apps/app_directory.c @@ -511,6 +511,11 @@ static struct ast_config *realtime_directory(char *context) const char *mailbox = ast_variable_retrieve(rtdata, category, "mailbox"); const char *ctx = ast_variable_retrieve(rtdata, category, "context"); + if (ast_strlen_zero(mailbox)) { + ast_debug(3, "Skipping result with missing or empty mailbox\n"); + continue; + } + fullname = ast_variable_retrieve(rtdata, category, "fullname"); hidefromdir = ast_variable_retrieve(rtdata, category, "hidefromdir"); if (ast_true(hidefromdir)) { @@ -531,7 +536,7 @@ static struct ast_config *realtime_directory(char *context) /* Does the context exist within the config file? If not, make one */ if (!(cat = ast_category_get(cfg, ctx, NULL))) { - if (!(cat = ast_category_new(ctx, "", 99999))) { + if (!(cat = ast_category_new_dynamic(ctx))) { ast_log(LOG_WARNING, "Out of memory\n"); ast_config_destroy(cfg); if (rtdata) { |