[FFmpeg-devel] [PATCH] libavfilter/vf_dnn_detect: bbox index may bigger than bbox number

[Guo, Yejun]

> -----Original Message-----
> From: ffmpeg-devel <ffmpeg-devel-bounces at ffmpeg.org> On Behalf Of
> wenbin.chen-at-intel.com at ffmpeg.org
> Sent: Monday, July 17, 2023 1:33 PM
> To: ffmpeg-devel at ffmpeg.org
> Subject: [FFmpeg-devel] [PATCH] libavfilter/vf_dnn_detect: bbox index may
> bigger than bbox number
> 
> From: Wenbin Chen <wenbin.chen at intel.com>
> 
> Fix a bug that queried bbox index may bigger than bbox's total number.
> 
> Signed-off-by: Wenbin Chen <wenbin.chen at intel.com>
> ---
>  libavfilter/vf_dnn_detect.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavfilter/vf_dnn_detect.c b/libavfilter/vf_dnn_detect.c
> index 06efce02a6..6ef04e0958 100644
> --- a/libavfilter/vf_dnn_detect.c
> +++ b/libavfilter/vf_dnn_detect.c
> @@ -106,12 +106,11 @@ static int dnn_detect_post_proc_ov(AVFrame
> *frame, DNNData *output, AVFilterCont
>          float x1     =      detections[i * detect_size + 5];
>          float y1     =      detections[i * detect_size + 6];
> 
> -        bbox = av_get_detection_bbox(header, i);
> -
>          if (conf < conf_threshold) {
>              continue;
>          }
> 
> +        bbox = av_get_detection_bbox(header, header->nb_bboxes -
> nb_bboxes);


Good catch, LGTM, will push soon, thanks.