[FFmpeg-cvslog] lavfi/dnn/dnn_io_proc.c: fix Improper use of negative value (NEGATIVE_RETURNS)

Guo, Yejun git at videolan.org
Tue May 18 04:51:02 EEST 2021


ffmpeg | branch: master | Guo, Yejun <yejun.guo at intel.com> | Tue May 11 13:46:49 2021 +0800| [3fb1d2e71c1c92ce282b373cd0319bfec56be2db] | committer: Guo, Yejun

lavfi/dnn/dnn_io_proc.c: fix Improper use of negative value (NEGATIVE_RETURNS)

fix coverity CID 1473511 and 1473566

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3fb1d2e71c1c92ce282b373cd0319bfec56be2db
---

 libavfilter/dnn/dnn_io_proc.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libavfilter/dnn/dnn_io_proc.c b/libavfilter/dnn/dnn_io_proc.c
index d5d2654162..02c8e13ed7 100644
--- a/libavfilter/dnn/dnn_io_proc.c
+++ b/libavfilter/dnn/dnn_io_proc.c
@@ -28,6 +28,9 @@ DNNReturnType ff_proc_from_dnn_to_frame(AVFrame *frame, DNNData *output, void *l
 {
     struct SwsContext *sws_ctx;
     int bytewidth = av_image_get_linesize(frame->format, frame->width, 0);
+    if (bytewidth < 0) {
+        return DNN_ERROR;
+    }
     if (output->dt != DNN_FLOAT) {
         avpriv_report_missing_feature(log_ctx, "data type rather than DNN_FLOAT");
         return DNN_ERROR;
@@ -98,6 +101,9 @@ static DNNReturnType proc_from_frame_to_dnn_frameprocessing(AVFrame *frame, DNND
 {
     struct SwsContext *sws_ctx;
     int bytewidth = av_image_get_linesize(frame->format, frame->width, 0);
+    if (bytewidth < 0) {
+        return DNN_ERROR;
+    }
     if (input->dt != DNN_FLOAT) {
         avpriv_report_missing_feature(log_ctx, "data type rather than DNN_FLOAT");
         return DNN_ERROR;



More information about the ffmpeg-cvslog mailing list