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

[Ronald S. Bultje]

Hi,

On Fri, Sep 17, 2010 at 5:03 PM, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> 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.

Applied a slightly modified version of the above.

Ronald