diff options
| author | Tilghman Lesher <tilghman@meg.abyt.es> | 2009-02-04 00:43:52 +0000 |
|---|---|---|
| committer | Tilghman Lesher <tilghman@meg.abyt.es> | 2009-02-04 00:43:52 +0000 |
| commit | f90021fdd0e95b4d6d857189573144f5ffa85240 (patch) | |
| tree | 79f6d1655cf4cecf61183b86cd0b5543b98f1a0f /main/pbx.c | |
| parent | f8f032e98bbe066ca6cccc59a34b75d73234eb19 (diff) | |
Ensure that commas placed in the middle of extension character classes do not
interfere with correct parsing of the extension. Also, if an unterminated
character class DOES make its way into the pbx core (through some other
method), ensure that it does not crash Asterisk.
(closes issue #14362)
Reported by: Nick_Lewis
Patches:
20090129__bug14362.diff.txt uploaded by Corydon76 (license 14)
Tested by: Corydon76
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@173311 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'main/pbx.c')
| -rw-r--r-- | main/pbx.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/main/pbx.c b/main/pbx.c index c6c29a2e8..39a3d8ba2 100644 --- a/main/pbx.c +++ b/main/pbx.c @@ -1900,6 +1900,9 @@ static struct match_char *add_exten_to_pattern_tree(struct ast_context *con, str *s2++ = s3; } s1++; s1++; + } else if (*s1 == '\0') { + ast_log(LOG_WARNING, "A matching ']' was not found for '[' in pattern string '%s'\n", extenbuf); + break; } else { *s2++ = *s1++; } |