Commit 49c8132b authored by Michael Niedermayer's avatar Michael Niedermayer

not checking error return, resulting double free

returning uninitalized pointers

Originally committed as revision 8002 to svn://svn.ffmpeg.org/ffmpeg/trunk
parent c861e7da
...@@ -227,7 +227,7 @@ static int process_ipmovie_chunk(IPMVEContext *s, ByteIOContext *pb, ...@@ -227,7 +227,7 @@ static int process_ipmovie_chunk(IPMVEContext *s, ByteIOContext *pb,
/* see if there are any pending packets */ /* see if there are any pending packets */
chunk_type = load_ipmovie_packet(s, pb, pkt); chunk_type = load_ipmovie_packet(s, pb, pkt);
if ((chunk_type == CHUNK_VIDEO) && (chunk_type != CHUNK_DONE)) if (chunk_type != CHUNK_DONE)
return chunk_type; return chunk_type;
/* read the next chunk, wherever the file happens to be pointing */ /* read the next chunk, wherever the file happens to be pointing */
...@@ -601,8 +601,10 @@ static int ipmovie_read_packet(AVFormatContext *s, ...@@ -601,8 +601,10 @@ static int ipmovie_read_packet(AVFormatContext *s,
ret = AVERROR_IO; ret = AVERROR_IO;
else if (ret == CHUNK_NOMEM) else if (ret == CHUNK_NOMEM)
ret = AVERROR_NOMEM; ret = AVERROR_NOMEM;
else else if (ret == CHUNK_VIDEO)
ret = 0; ret = 0;
else
ret = -1;
return ret; return ret;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment