diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-08-06 03:30:23 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-08-06 03:30:23 +0000 |
commit | 5b4f04081bb76ad28d20ef0ae2fa4570ca544a7d (patch) | |
tree | cb85f66463d15ccc1e37b30b51306f0fe80f642a /main | |
parent | 3ff9d7ccdfab16190eccb1ec7a4d2eaab0c53d3e (diff) |
Merged revisions 135938 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
................
r135938 | tilghman | 2008-08-05 22:29:42 -0500 (Tue, 05 Aug 2008) | 12 lines
Merged revisions 135915 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r135915 | tilghman | 2008-08-05 22:24:56 -0500 (Tue, 05 Aug 2008) | 4 lines
Since powerof() can return an error condition, it's foolhardy not to detect and
deal with that condition.
(Related to issue #13240)
........
................
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.0@135942 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main')
-rw-r--r-- | main/translate.c | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/main/translate.c b/main/translate.c index 750b2f5b6..eeeb7cbd1 100644 --- a/main/translate.c +++ b/main/translate.c @@ -265,7 +265,12 @@ struct ast_trans_pvt *ast_translator_build_path(int dest, int source) source = powerof(source); dest = powerof(dest); - + + if (source == -1 || dest == -1) { + ast_log(LOG_WARNING, "No translator path: (%s codec is not valid)\n", source == -1 ? "starting" : "ending"); + return NULL; + } + AST_RWLIST_RDLOCK(&translators); while (source != dest) { @@ -625,6 +630,10 @@ int __ast_register_translator(struct ast_translator *t, struct ast_module *mod) t->dstfmt = powerof(t->dstfmt); t->active = 1; + if (t->srcfmt == -1 || t->dstfmt == -1) { + ast_log(LOG_WARNING, "Invalid translator path: (%s codec is not valid)\n", t->srcfmt == -1 ? "starting" : "ending"); + return -1; + } if (t->plc_samples) { if (t->buffer_samples < t->plc_samples) { ast_log(LOG_WARNING, "plc_samples %d buffer_samples %d\n", @@ -794,6 +803,10 @@ unsigned int ast_translate_path_steps(unsigned int dest, unsigned int src) src = powerof(src); dest = powerof(dest); + if (src == -1 || dest == -1) { + ast_log(LOG_WARNING, "No translator path: (%s codec is not valid)\n", src == -1 ? "starting" : "ending"); + return -1; + } AST_RWLIST_RDLOCK(&translators); if (tr_matrix[src][dest].step) |