* [FFmpeg-devel] [PATCH v1] fftools/ffmpeg_enc: NULL check for frame pointer removed [not found] <CGME20231006120550eucas1p1b4e622e5acbcf5c18d93ea8a7c58d314@eucas1p1.samsung.com> @ 2023-10-06 12:05 ` Dawid Kozinski 2023-10-06 12:06 ` Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics 0 siblings, 1 reply; 2+ messages in thread From: Dawid Kozinski @ 2023-10-06 12:05 UTC (permalink / raw) To: d.frankiewic, ffmpeg-devel; +Cc: Dawid Kozinski Signed-off-by: Dawid Kozinski <d.kozinski@samsung.com> --- fftools/ffmpeg_enc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fftools/ffmpeg_enc.c b/fftools/ffmpeg_enc.c index ec9cebbd96..61fd31823e 100644 --- a/fftools/ffmpeg_enc.c +++ b/fftools/ffmpeg_enc.c @@ -951,8 +951,8 @@ early_exit: if (debug_ts) { av_log(NULL, AV_LOG_INFO, "filter -> pts:%s pts_time:%s exact:%f time_base:%d/%d\n", - frame ? av_ts2str(frame->pts) : "NULL", - (enc && frame) ? av_ts2timestr(frame->pts, &enc->time_base) : "NULL", + av_ts2str(frame->pts), + (enc) ? av_ts2timestr(frame->pts, &enc->time_base) : "NULL", float_pts, enc ? enc->time_base.num : -1, enc ? enc->time_base.den : -1); -- 2.25.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [FFmpeg-devel] [PATCH v1] fftools/ffmpeg_enc: NULL check for frame pointer removed 2023-10-06 12:05 ` [FFmpeg-devel] [PATCH v1] fftools/ffmpeg_enc: NULL check for frame pointer removed Dawid Kozinski @ 2023-10-06 12:06 ` Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics 0 siblings, 0 replies; 2+ messages in thread From: Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics @ 2023-10-06 12:06 UTC (permalink / raw) To: d.frankiewic, ffmpeg-devel The pointer 'frame', which is dereferenced at ffmpeg_enc.c:930, is compared to a NULL value at ffmpeg_enc.c:955. The pointer 'frame' is checked in the video_sync_process() function before calling the adjust_frame_pts_to_encoder_tb function > -----Original Message----- > From: Dawid Kozinski <d.kozinski@samsung.com> > Sent: piątek, 6 października 2023 14:06 > To: d.frankiewic@samsung.com; ffmpeg-devel@ffmpeg.org > Cc: Dawid Kozinski <d.kozinski@samsung.com> > Subject: [PATCH v1] fftools/ffmpeg_enc: NULL check for frame pointer removed > > Signed-off-by: Dawid Kozinski <d.kozinski@samsung.com> > --- > fftools/ffmpeg_enc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fftools/ffmpeg_enc.c b/fftools/ffmpeg_enc.c index > ec9cebbd96..61fd31823e 100644 > --- a/fftools/ffmpeg_enc.c > +++ b/fftools/ffmpeg_enc.c > @@ -951,8 +951,8 @@ early_exit: > > if (debug_ts) { > av_log(NULL, AV_LOG_INFO, "filter -> pts:%s pts_time:%s exact:%f > time_base:%d/%d\n", > - frame ? av_ts2str(frame->pts) : "NULL", > - (enc && frame) ? av_ts2timestr(frame->pts, &enc->time_base) : > "NULL", > + av_ts2str(frame->pts), > + (enc) ? av_ts2timestr(frame->pts, &enc->time_base) : > + "NULL", > float_pts, > enc ? enc->time_base.num : -1, > enc ? enc->time_base.den : -1); > -- > 2.25.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-10-06 12:07 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20231006120550eucas1p1b4e622e5acbcf5c18d93ea8a7c58d314@eucas1p1.samsung.com>
2023-10-06 12:05 ` [FFmpeg-devel] [PATCH v1] fftools/ffmpeg_enc: NULL check for frame pointer removed Dawid Kozinski
2023-10-06 12:06 ` Dawid Kozinski/Multimedia (PLT) /SRPOL/Staff Engineer/Samsung Electronics
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git