[FFmpeg-devel] [PATCH 05/17] avformat/webm_chunk: Don't use child AVFormatContext for logging

Andreas Rheinhardt andreas.rheinhardt at gmail.com
Thu Mar 26 03:43:29 EET 2020


Andreas Rheinhardt:
> Andreas Rheinhardt:
>> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>
>> ---
>>  libavformat/webm_chunk.c | 7 +++----
>>  1 file changed, 3 insertions(+), 4 deletions(-)
>>
>> diff --git a/libavformat/webm_chunk.c b/libavformat/webm_chunk.c
>> index bc3d346a00..0d4e3598ed 100644
>> --- a/libavformat/webm_chunk.c
>> +++ b/libavformat/webm_chunk.c
>> @@ -87,25 +87,24 @@ static int chunk_mux_init(AVFormatContext *s)
>>  static int get_chunk_filename(AVFormatContext *s, int is_header, char filename[MAX_FILENAME_SIZE])
>>  {
>>      WebMChunkContext *wc = s->priv_data;
>> -    AVFormatContext *oc = wc->avf;
>>      if (!filename) {
>>          return AVERROR(EINVAL);
>>      }
>>      if (is_header) {
>>          int len;
>>          if (!wc->header_filename) {
>> -            av_log(oc, AV_LOG_ERROR, "No header filename provided\n");
>> +            av_log(s, AV_LOG_ERROR, "No header filename provided\n");
>>              return AVERROR(EINVAL);
>>          }
>>          len = av_strlcpy(filename, wc->header_filename, MAX_FILENAME_SIZE);
>>          if (len >= MAX_FILENAME_SIZE) {
>> -            av_log(oc, AV_LOG_ERROR, "Header filename too long\n");
>> +            av_log(s, AV_LOG_ERROR, "Header filename too long\n");
>>              return AVERROR(EINVAL);
>>          }
>>      } else {
>>          if (av_get_frame_filename(filename, MAX_FILENAME_SIZE,
>>                                    s->url, wc->chunk_index - 1) < 0) {
>> -            av_log(oc, AV_LOG_ERROR, "Invalid chunk filename template '%s'\n", s->url);
>> +            av_log(s, AV_LOG_ERROR, "Invalid chunk filename template '%s'\n", s->url);
>>              return AVERROR(EINVAL);
>>          }
>>      }
>>
> Ping. If no one objects, I will merge the webm_chunk patches tomorrow.
> 
> - Andreas
> 
Pushed.

- Andreas


More information about the ffmpeg-devel mailing list