diff options
| author | Joshua Colp <jcolp@digium.com> | 2017-10-24 13:57:06 -0500 |
|---|---|---|
| committer | Gerrit Code Review <gerrit2@gerrit.digium.api> | 2017-10-24 13:57:06 -0500 |
| commit | cad907933987c2639555336c556751340e93bcd6 (patch) | |
| tree | 9afecbed5f684e959344420d976828bd5bcd400f | |
| parent | 558431635c6e7e808d0c20ce186a20223ee3206f (diff) | |
| parent | ddce3ddab5c781f2410fec718744eaee929e055c (diff) | |
Merge "test_config: Fix failure and segfault when config_hook is run twice." into 15
| -rw-r--r-- | tests/test_config.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/tests/test_config.c b/tests/test_config.c index 8675cb890..d74726a1a 100644 --- a/tests/test_config.c +++ b/tests/test_config.c @@ -957,7 +957,8 @@ AST_TEST_DEFINE(config_hook) { enum ast_test_result_state res = AST_TEST_FAIL; enum config_hook_flags hook_flags = { 0, }; - struct ast_flags config_flags = { CONFIG_FLAG_FILEUNCHANGED }; + struct ast_flags config_flags = { 0 }; + struct ast_flags reload_flags = { CONFIG_FLAG_FILEUNCHANGED }; struct ast_config *cfg; switch (cmd) { @@ -1025,7 +1026,7 @@ AST_TEST_DEFINE(config_hook) * Hook should not run */ hook_run = 0; - cfg = ast_config_load(CONFIG_FILE, config_flags); + cfg = ast_config_load(CONFIG_FILE, reload_flags); /* Only destroy this cfg conditionally. Otherwise a crash happens. */ if (cfg != CONFIG_STATUS_FILEUNCHANGED) { ast_config_destroy(cfg); |