[FFmpeg-devel] [PATCH 1/2] lavfi/dnn_backend_openvino.c: Fix Memory Leak in execute_model_ov

Shubhanshu Saxena shubhanshu.e01 at gmail.com
Sun Jul 11 19:15:01 EEST 2021


In cases where the execution inside the function execute_model_ov fails,
the OVRequestItem must be pushed back to the request_queue before returning
the error. In case pushing back fails, release the allocated memory.

Signed-off-by: Shubhanshu Saxena <shubhanshu.e01 at gmail.com>
---
 libavfilter/dnn/dnn_backend_openvino.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/libavfilter/dnn/dnn_backend_openvino.c b/libavfilter/dnn/dnn_backend_openvino.c
index f34b8150f5..b340859c12 100644
--- a/libavfilter/dnn/dnn_backend_openvino.c
+++ b/libavfilter/dnn/dnn_backend_openvino.c
@@ -432,6 +432,8 @@ static DNNReturnType execute_model_ov(RequestItem *request, Queue *inferenceq)
     OVModel *ov_model;
 
     if (ff_queue_size(inferenceq) == 0) {
+        ie_infer_request_free(&request->infer_request);
+        av_freep(&request);
         return DNN_SUCCESS;
     }
 
@@ -443,7 +445,7 @@ static DNNReturnType execute_model_ov(RequestItem *request, Queue *inferenceq)
     if (task->async) {
         ret = fill_model_input_ov(ov_model, request);
         if (ret != DNN_SUCCESS) {
-            return ret;
+            goto err;
         }
         status = ie_infer_set_completion_callback(request->infer_request, &request->callback);
         if (status != OK) {
@@ -459,7 +461,7 @@ static DNNReturnType execute_model_ov(RequestItem *request, Queue *inferenceq)
     } else {
         ret = fill_model_input_ov(ov_model, request);
         if (ret != DNN_SUCCESS) {
-            return ret;
+            goto err;
         }
         status = ie_infer_request_infer(request->infer_request);
         if (status != OK) {
-- 
2.25.1



More information about the ffmpeg-devel mailing list