[Libva] [libva][libva-intel-driver][PATCH] Fix segmentation fault caused by 4711da5

Zhao, Yakui yakui.zhao at intel.com
Thu Jan 8 20:54:25 PST 2015


On Thu, 2015-01-08 at 21:18 -0700, Xiang, Haihao wrote:
> The corresponding pointer is NULL if VAProcFilterSkinToneEnhancement
> isn't enabled in the pipeline

LGTM. 
> 
> Signed-off-by: Xiang, Haihao <haihao.xiang at intel.com>
> ---
>  src/gen75_vpp_vebox.c | 7 ++++---
>  1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/src/gen75_vpp_vebox.c b/src/gen75_vpp_vebox.c
> index b541b32..3eb5040 100644
> --- a/src/gen75_vpp_vebox.c
> +++ b/src/gen75_vpp_vebox.c
> @@ -565,13 +565,14 @@ void set_std_table_9(struct intel_vebox_context *proc_ctx, unsigned int *p_table
>  void hsw_veb_iecp_std_table(VADriverContextP ctx, struct intel_vebox_context *proc_ctx)
>  {
>      unsigned int *p_table = proc_ctx->iecp_state_table.ptr + 0 ;
> -    int stde_factor = 0; //default value
> -    VAProcFilterParameterBuffer * std_param = (VAProcFilterParameterBuffer *) proc_ctx->filter_iecp_std;
> -    stde_factor = std_param->value;
>  
>      if(!(proc_ctx->filters_mask & VPP_IECP_STD_STE)){ 
>          memset(p_table, 0, 29 * 4);
>      }else{
> +        int stde_factor = 0; //default value
> +        VAProcFilterParameterBuffer * std_param = (VAProcFilterParameterBuffer *) proc_ctx->filter_iecp_std;
> +        stde_factor = std_param->value;
> +
>          //DWord 0
>          *p_table ++ = ( 154 << 24 |   // V_Mid
>                          110 << 16 |   // U_Mid




More information about the Libva mailing list