[FFmpeg-devel] [Patch] MMSH handle get_chunk_header error inget_http_header_data

Ronald S. Bultje rsbultje
Fri Sep 17 23:03:15 CEST 2010


Hi,

On Fri, Sep 17, 2010 at 4:05 PM, Stephen D'Angelo <SDAngelo at evertz.com> wrote:
> Sorry, the above patch contained extra stuff in it. ?Here is the correct
> patch.
[..]
> +        res = get_chunk_header(mmsh, &len);
> +        if (res < 0) {
> +            return;
> +        }
> +        chunk_type = res;

+ res = chunk_type = ..;
+ if (res < 0) {
+     return res; <- not return;!
+ }

Is one line less, and actually returns the error code.

Ronald



More information about the ffmpeg-devel mailing list