From 3123e81c862ddc2fd57df873e54da6ee8f9fa675 Mon Sep 17 00:00:00 2001 From: Ram Mohan M Date: Mon, 30 Oct 2023 17:09:40 +0530 Subject: [PATCH] libavcenc: validate dimensions after applying config params Before applying the config params (dimensions), the api is validating the input buffer dimensions. This causes encode fails wrongly. This is corrected. --- encoder/ih264e_api.c | 103 --------------------------------- encoder/ih264e_encode.c | 124 ++++++++++++++++++++++++++++++++++++---- 2 files changed, 114 insertions(+), 113 deletions(-) diff --git a/encoder/ih264e_api.c b/encoder/ih264e_api.c index f76fbd1c..019a2880 100644 --- a/encoder/ih264e_api.c +++ b/encoder/ih264e_api.c @@ -135,101 +135,6 @@ /* Function Definitions */ /*****************************************************************************/ -/** -******************************************************************************* -* -* @brief -* Used to test validity of input dimensions -* -* @par Description: -* Dimensions of the input buffer passed to encode call are validated -* -* @param[in] ps_codec -* Codec context -* -* @param[in] ps_ip -* Pointer to input structure -* -* @param[out] ps_op -* Pointer to output structure -* -* @returns error status -* -* @remarks none -* -******************************************************************************* -*/ -static IV_STATUS_T api_check_input_dimensions(codec_t *ps_codec, - ih264e_video_encode_ip_t *ps_ip, - ih264e_video_encode_op_t *ps_op) -{ - UWORD32 u4_wd, u4_ht; - cfg_params_t *ps_curr_cfg = &ps_codec->s_cfg; - iv_raw_buf_t *ps_inp_buf = &ps_ip->s_ive_ip.s_inp_buf; - - u4_wd = ps_inp_buf->au4_wd[0]; - u4_ht = ps_inp_buf->au4_ht[0]; - switch (ps_inp_buf->e_color_fmt) - { - case IV_YUV_420P: - if (((ps_inp_buf->au4_wd[0] / 2) != ps_inp_buf->au4_wd[1]) || - ((ps_inp_buf->au4_wd[0] / 2) != ps_inp_buf->au4_wd[2]) || - (ps_inp_buf->au4_wd[1] != ps_inp_buf->au4_wd[2])) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_WIDTH_NOT_SUPPORTED; - return (IV_FAIL); - } - if (((ps_inp_buf->au4_ht[0] / 2) != ps_inp_buf->au4_ht[1]) || - ((ps_inp_buf->au4_ht[0] / 2) != ps_inp_buf->au4_ht[2]) || - (ps_inp_buf->au4_ht[1] != ps_inp_buf->au4_ht[2])) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_HEIGHT_NOT_SUPPORTED; - return (IV_FAIL); - } - break; - case IV_YUV_420SP_UV: - case IV_YUV_420SP_VU: - if (ps_inp_buf->au4_wd[0] != ps_inp_buf->au4_wd[1]) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_WIDTH_NOT_SUPPORTED; - return (IV_FAIL); - } - if ((ps_inp_buf->au4_ht[0] / 2) != ps_inp_buf->au4_ht[1]) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_HEIGHT_NOT_SUPPORTED; - return (IV_FAIL); - } - break; - case IV_YUV_422ILE: - u4_wd = ps_inp_buf->au4_wd[0] / 2; - break; - default: - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_INPUT_CHROMA_FORMAT_NOT_SUPPORTED; - return (IV_FAIL); - } - - if (u4_wd != ps_curr_cfg->u4_disp_wd) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_WIDTH_NOT_SUPPORTED; - return (IV_FAIL); - } - - if (u4_ht != ps_curr_cfg->u4_disp_ht) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IH264E_HEIGHT_NOT_SUPPORTED; - return (IV_FAIL); - } - - return IV_SUCCESS; -} - /** ******************************************************************************* * @@ -930,14 +835,6 @@ static IV_STATUS_T api_check_struct_sanity(iv_obj_t *ps_handle, return (IV_FAIL); } - if (NULL != ps_ip->s_ive_ip.s_inp_buf.apv_bufs[0] && - ps_codec->i4_header_mode != 1 && - IV_SUCCESS != api_check_input_dimensions(ps_codec, ps_ip, ps_op)) - { - ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; - ps_op->s_ive_op.u4_error_code |= IVE_ERR_OP_ENCODE_API_STRUCT_SIZE_INCORRECT; - return (IV_FAIL); - } break; } diff --git a/encoder/ih264e_encode.c b/encoder/ih264e_encode.c index f5d14d22..e6ca3be6 100644 --- a/encoder/ih264e_encode.c +++ b/encoder/ih264e_encode.c @@ -183,6 +183,101 @@ IH264E_ERROR_T ih264e_wait_for_thread(UWORD32 sleep_us) return IH264E_SUCCESS; } +/** +******************************************************************************* +* +* @brief +* Used to test validity of input dimensions +* +* @par Description: +* Dimensions of the input buffer passed to encode call are validated +* +* @param[in] ps_codec +* Codec context +* +* @param[in] ps_ip +* Pointer to input structure +* +* @param[out] ps_op +* Pointer to output structure +* +* @returns error status +* +* @remarks none +* +******************************************************************************* +*/ +static IV_STATUS_T api_check_input_dimensions(codec_t *ps_codec, + ih264e_video_encode_ip_t *ps_ip, + ih264e_video_encode_op_t *ps_op) +{ + UWORD32 u4_wd, u4_ht; + cfg_params_t *ps_curr_cfg = &ps_codec->s_cfg; + iv_raw_buf_t *ps_inp_buf = &ps_ip->s_ive_ip.s_inp_buf; + + u4_wd = ps_inp_buf->au4_wd[0]; + u4_ht = ps_inp_buf->au4_ht[0]; + switch (ps_inp_buf->e_color_fmt) + { + case IV_YUV_420P: + if (((ps_inp_buf->au4_wd[0] / 2) != ps_inp_buf->au4_wd[1]) || + ((ps_inp_buf->au4_wd[0] / 2) != ps_inp_buf->au4_wd[2]) || + (ps_inp_buf->au4_wd[1] != ps_inp_buf->au4_wd[2])) + { + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_WIDTH_NOT_SUPPORTED; + return (IV_FAIL); + } + if (((ps_inp_buf->au4_ht[0] / 2) != ps_inp_buf->au4_ht[1]) || + ((ps_inp_buf->au4_ht[0] / 2) != ps_inp_buf->au4_ht[2]) || + (ps_inp_buf->au4_ht[1] != ps_inp_buf->au4_ht[2])) + { + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_HEIGHT_NOT_SUPPORTED; + return (IV_FAIL); + } + break; + case IV_YUV_420SP_UV: + case IV_YUV_420SP_VU: + if (ps_inp_buf->au4_wd[0] != ps_inp_buf->au4_wd[1]) + { + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_WIDTH_NOT_SUPPORTED; + return (IV_FAIL); + } + if ((ps_inp_buf->au4_ht[0] / 2) != ps_inp_buf->au4_ht[1]) + { + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_HEIGHT_NOT_SUPPORTED; + return (IV_FAIL); + } + break; + case IV_YUV_422ILE: + u4_wd = ps_inp_buf->au4_wd[0] / 2; + break; + default: + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_INPUT_CHROMA_FORMAT_NOT_SUPPORTED; + return (IV_FAIL); + } + + if (u4_wd != ps_curr_cfg->u4_disp_wd) + { + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_WIDTH_NOT_SUPPORTED; + return (IV_FAIL); + } + + if (u4_ht != ps_curr_cfg->u4_disp_ht) + { + ps_op->s_ive_op.u4_error_code |= 1 << IVE_UNSUPPORTEDPARAM; + ps_op->s_ive_op.u4_error_code |= IH264E_HEIGHT_NOT_SUPPORTED; + return (IV_FAIL); + } + + return IV_SUCCESS; +} + /** ****************************************************************************** * @@ -427,24 +522,33 @@ WORD32 ih264e_encode(iv_obj_t *ps_codec_obj, void *pv_api_ip, void *pv_api_op) } } #endif - /****************************************************************** - * INSERT LOGO - *****************************************************************/ -#ifdef LOGO_EN - if (s_inp_buf.s_raw_buf.apv_bufs[0] != NULL && + + if (ps_video_encode_ip->s_ive_ip.s_inp_buf.apv_bufs[0] != NULL && ps_codec->i4_header_mode != 1) { - ih264e_insert_logo(s_inp_buf.s_raw_buf.apv_bufs[0], - s_inp_buf.s_raw_buf.apv_bufs[1], - s_inp_buf.s_raw_buf.apv_bufs[2], - s_inp_buf.s_raw_buf.au4_strd[0], + if (IV_SUCCESS != api_check_input_dimensions(ps_codec, pv_api_ip, pv_api_op)) + { + error_status = IVE_ERR_OP_ENCODE_API_STRUCT_SIZE_INCORRECT; + SET_ERROR_ON_RETURN(error_status, + IVE_FATALERROR, + ps_video_encode_op->s_ive_op.u4_error_code, + IV_FAIL); + } + /****************************************************************** + * INSERT LOGO + *****************************************************************/ +#ifdef LOGO_EN + ih264e_insert_logo(ps_video_encode_ip->s_ive_ip.s_inp_buf.apv_bufs[0], + ps_video_encode_ip->s_ive_ip.s_inp_buf.apv_bufs[1], + ps_video_encode_ip->s_ive_ip.s_inp_buf.apv_bufs[2], + ps_video_encode_ip->s_ive_ip.s_inp_buf.au4_strd[0], 0, 0, ps_codec->s_cfg.e_inp_color_fmt, ps_codec->s_cfg.u4_disp_wd, ps_codec->s_cfg.u4_disp_ht); - } #endif /*LOGO_EN*/ + } /* In case of alt ref and B pics we will have non reference frame in stream */ if (ps_codec->s_cfg.u4_enable_alt_ref || ps_codec->s_cfg.u4_num_bframes)