diff options
author | Richard Mudgett <rmudgett@digium.com> | 2017-10-25 14:38:19 -0500 |
---|---|---|
committer | Richard Mudgett <rmudgett@digium.com> | 2017-10-26 11:48:23 -0500 |
commit | ca7f3d297b53785ba3b5282a569ea8add9717c8b (patch) | |
tree | 77f40650715a253cc5c944dac8193faf94297a07 | |
parent | 28368b355344ea1001ab2594905109e1090c0704 (diff) |
codec.c: Defensively check the returned samples.
Earlier versions of the codec_opus samples_count callback can return
negative error values on undecodable frames. This resulted in a divide by
zero exception.
* Added a defensive check in ast_codec_samples_count() for a "negative"
samples count return value. Log the event and set the count to zero.
ASTERISK-27194
Change-Id: Icf69350307ecbbc80a3d74de46af9bd80ea17819
-rw-r--r-- | main/codec.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/main/codec.c b/main/codec.c index c253233bb..d0e63682b 100644 --- a/main/codec.c +++ b/main/codec.c @@ -393,6 +393,11 @@ unsigned int ast_codec_samples_count(struct ast_frame *frame) if (codec->samples_count) { samples = codec->samples_count(frame); + if ((int) samples < 0) { + ast_log(LOG_WARNING, "Codec %s returned invalid number of samples.\n", + ast_format_get_name(frame->subclass.format)); + samples = 0; + } } else { ast_log(LOG_WARNING, "Unable to calculate samples for codec %s\n", ast_format_get_name(frame->subclass.format)); |