[SCM] libav/experimental: Check return value of get_chunk_header(). Since enum can be unsigned, the current code wouldn't always error out on errors.
siretart at users.alioth.debian.org
siretart at users.alioth.debian.org
Sun Jun 30 17:16:35 UTC 2013
The following commit has been merged in the experimental branch:
commit 3dedbeff7b74d7820b4395c87614edfa8e688fa4
Author: Ronald S. Bultje <rsbultje at gmail.com>
Date: Wed Sep 29 15:43:36 2010 +0000
Check return value of get_chunk_header(). Since enum can be unsigned, the
current code wouldn't always error out on errors.
Based on patch by Stephen d'Angelo <sdangelo evertz com>.
Originally committed as revision 25260 to svn://svn.ffmpeg.org/ffmpeg/trunk
diff --git a/libavformat/mmsh.c b/libavformat/mmsh.c
index dbbaab6..3581431 100644
--- a/libavformat/mmsh.c
+++ b/libavformat/mmsh.c
@@ -147,9 +147,9 @@ static int get_http_header_data(MMSHContext *mmsh)
for (;;) {
len = 0;
- chunk_type = get_chunk_header(mmsh, &len);
- if (chunk_type < 0) {
- return chunk_type;
+ res = chunk_type = get_chunk_header(mmsh, &len);
+ if (res < 0) {
+ return res;
} else if (chunk_type == CHUNK_TYPE_ASF_HEADER){
// get asf header and stored it
if (!mms->header_parsed) {
--
Libav/FFmpeg packaging
More information about the pkg-multimedia-commits
mailing list