[vlc-devel] [vlc-commits] Fixing HLS on win32.

[Rafaël Carré]

Le 2012-03-04 17:05, Hugo Beauzée-Luyssen a écrit :
> On Sun, Mar 4, 2012 at 10:13 PM, Rafaël Carré <funman at videolan.org> wrote:
>> Le 2012-03-04 15:51, Hugo Beauzée-Luyssen a écrit :
>>> vlc | branch: master | Hugo Beauzée-Luyssen <beauze.h at gmail.com> | Sun Mar  4 21:50:45 2012 +0100| [7573128aa74cd02d78269c8ff9d9aa30337ef859] | committer: Hugo Beauzée-Luyssen
>>>
>>> Fixing HLS on win32.
>>>
>>>> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=7573128aa74cd02d78269c8ff9d9aa30337ef859
>>> ---
>>>
>>>  modules/stream_filter/httplive.c |   10 ++++++++++
>>>  1 files changed, 10 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/modules/stream_filter/httplive.c b/modules/stream_filter/httplive.c
>>> index 984a390..224e312 100644
>>> --- a/modules/stream_filter/httplive.c
>>> +++ b/modules/stream_filter/httplive.c
>>> @@ -1844,6 +1844,16 @@ static int Open(vlc_object_t *p_this)
>>>      }
>>>      p_sys->m3u8 = psz_uri;
>>>
>>> +    char *new_path;
>>> +    if (asprintf(&new_path, "%s.ts", s->psz_path) < 0)
>>> +    {
>>> +        free(p_sys->m3u8);
>>> +        free(p_sys);
>>> +        return VLC_ENOMEM;
>>> +    }
>>> +    free(s->psz_path);
>>
>> This one isn't freed in the error path
>>
> It only has to be freed when it's overwritten, so no issue here.

OK, wasn't sure about it.