From dc3c692e4ce7e6b1d75d50d47b51cb14e7b1ac02 Mon Sep 17 00:00:00 2001
From: Herman Chen <herman.chen@rock-chips.com>
Date: Wed, 9 Nov 2022 10:06:22 +0800
Subject: [PATCH] [mpp_enc]: Add fps check for negtive value

Signed-off-by: Herman Chen <herman.chen@rock-chips.com>
Change-Id: I7bad72c3fcebb16d9ed03e58a3c45b7c4c9a4948
---
 mpp/codec/mpp_enc_impl.cpp | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/mpp/codec/mpp_enc_impl.cpp b/mpp/codec/mpp_enc_impl.cpp
index 3bd218eb..955cfdb4 100644
--- a/mpp/codec/mpp_enc_impl.cpp
+++ b/mpp/codec/mpp_enc_impl.cpp
@@ -676,6 +676,15 @@ MPP_RET mpp_enc_proc_rc_cfg(MppCodingType coding, MppEncRcCfg *dst, MppEncRcCfg
                 ret = MPP_ERR_VALUE;
             }
         }
+
+        if (dst->fps_in_num < 0 || dst->fps_in_denorm < 0 ||
+            dst->fps_out_num < 0 || dst->fps_out_denorm < 0) {
+            mpp_err("invalid fps cfg [number:denorm:flex]: in [%d:%d:%d] out [%d:%d:%d]\n",
+                    dst->fps_in_num, dst->fps_in_denorm, dst->fps_in_flex,
+                    dst->fps_out_num, dst->fps_out_denorm, dst->fps_out_flex);
+            ret = MPP_ERR_VALUE;
+        }
+
         // if I frame min/max is not set use normal case
         if (dst->qp_min_i <= 0)
             dst->qp_min_i = dst->qp_min;