[FFmpeg-devel] [PATCH] avformat/hlsenc: compiler warning fix

Bodecs Bela bodecsb at vivanet.hu
Mon Jan 20 14:56:06 EET 2020


2020.01.20. 13:46 keltezéssel, Carl Eugen Hoyos írta:
> Am Mo., 20. Jan. 2020 um 13:18 Uhr schrieb Bodecs Bela <bodecsb at vivanet.hu>:
>>
>> 2020.01.20. 13:07 keltezéssel, Carl Eugen Hoyos írta:
>>> Am Mo., 20. Jan. 2020 um 09:41 Uhr schrieb Bodecs Bela <bodecsb at vivanet.hu>:
>>>> 2020.01.20. 8:46 keltezéssel, Andreas Rheinhardt írta:
>>>>> On Mon, Jan 20, 2020 at 8:35 AM Bodecs Bela <bodecsb at vivanet.hu> wrote:
>>>>>
>>>>>> Dear All,
>>>>>>
>>>>>> This patch corrects compiler warning about returning possibly uninitialized
>>>>>> variable in hls_write_header function.
>>>>>>
>>>>>> Which compiler warns about this?
>>>> libavformat/hlsenc.c: In function ‘hls_write_header’:
>>>> libavformat/hlsenc.c:2174: warning: ‘ret’ may be used uninitialized in
>>>> this function
>>> Looks like a false positive, does not warn with newer compilers.
>> yes and no.
>>
>> no, because if hls->nb_varstreams == 0, then for loop body never
>> executes and so ret does not get any value.
>>
>> yes, because there is no chance for hls->nb_varstreams to be zero.
>>
>> so, I think setting ret to 0 at its declaration does not do any problem
>> and logically right.
> No, update your compiler instead.
which of my statements did your "No" reference?
>
> Carl Eugen
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".


More information about the ffmpeg-devel mailing list