diff mbox series

[FFmpeg-devel,2/2] lavfi/vf_drawtext.c: fix CID 1485003

Message ID 20210604022246.5193-2-ting.fu@intel.com
State Accepted
Commit e01bf559dfaeaadac8112af4ad4689901a344b92
Headers show
Series [FFmpeg-devel,1/2] lavfi/vf_drawbox.c: fix CID 1485004 | expand

Checks

Context Check Description
andriy/x86_make success Make finished
andriy/x86_make_fate success Make fate finished
andriy/PPC64_make success Make finished
andriy/PPC64_make_fate success Make fate finished

Commit Message

Fu, Ting June 4, 2021, 2:22 a.m. UTC
CID 1485003: Memory - illegal accesses (UNINIT)
Using uninitialized value "sd".

Signed-off-by: Ting Fu <ting.fu@intel.com>
---
 libavfilter/vf_drawtext.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Guo, Yejun June 9, 2021, 1:27 a.m. UTC | #1
> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Ting
> Fu
> Sent: 2021年6月4日 10:23
> To: ffmpeg-devel@ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH 2/2] lavfi/vf_drawtext.c: fix CID 1485003
> 
> CID 1485003: Memory - illegal accesses (UNINIT)
> Using uninitialized value "sd".
> 
> Signed-off-by: Ting Fu <ting.fu@intel.com>
> ---
>  libavfilter/vf_drawtext.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavfilter/vf_drawtext.c b/libavfilter/vf_drawtext.c
> index 382d589e26..c4c09894e4 100644
> --- a/libavfilter/vf_drawtext.c
> +++ b/libavfilter/vf_drawtext.c
> @@ -1554,7 +1554,7 @@ static int filter_frame(AVFilterLink *inlink,
> AVFrame *frame)
>      AVFrameSideData *sd;
>      int loop = 1;
> 
> -    if (s->text_source == AV_FRAME_DATA_DETECTION_BBOXES && sd) {
> +    if (s->text_source == AV_FRAME_DATA_DETECTION_BBOXES) {
>          sd = av_frame_get_side_data(frame,
> AV_FRAME_DATA_DETECTION_BBOXES);
>          if (sd) {
>              header = (AVDetectionBBoxHeader *)sd->data;
> --
thanks, will push soon.
diff mbox series

Patch

diff --git a/libavfilter/vf_drawtext.c b/libavfilter/vf_drawtext.c
index 382d589e26..c4c09894e4 100644
--- a/libavfilter/vf_drawtext.c
+++ b/libavfilter/vf_drawtext.c
@@ -1554,7 +1554,7 @@  static int filter_frame(AVFilterLink *inlink, AVFrame *frame)
     AVFrameSideData *sd;
     int loop = 1;
 
-    if (s->text_source == AV_FRAME_DATA_DETECTION_BBOXES && sd) {
+    if (s->text_source == AV_FRAME_DATA_DETECTION_BBOXES) {
         sd = av_frame_get_side_data(frame, AV_FRAME_DATA_DETECTION_BBOXES);
         if (sd) {
             header = (AVDetectionBBoxHeader *)sd->data;