Re: [Mesa-dev] [PATCH v3 2/4] intel/decoder: Avoid freeing invalid pointer

2018-09-13 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 06/09/2018 20:37, Sagar Ghuge wrote: v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin) v3: Handle case where genxml is empty (Lionel Landwerlin) Signed-off-by: Sagar Ghuge --- src/intel/common/gen_decoder.c | 18 +- 1 fil

Re: [Mesa-dev] [PATCH v3 2/4] intel/decoder: Avoid freeing invalid pointer

2018-09-13 Thread Sagar Ghuge
Ping. On 9/6/18 12:37 PM, Sagar Ghuge wrote: > v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin) > > v3: Handle case where genxml is empty (Lionel Landwerlin) > > Signed-off-by: Sagar Ghuge > --- > src/intel/common/gen_decoder.c | 18 +- > 1 file changed, 13 i

[Mesa-dev] [PATCH v3 2/4] intel/decoder: Avoid freeing invalid pointer

2018-09-06 Thread Sagar Ghuge
v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin) v3: Handle case where genxml is empty (Lionel Landwerlin) Signed-off-by: Sagar Ghuge --- src/intel/common/gen_decoder.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/src/intel/common/ge

Re: [Mesa-dev] [PATCH v3 2/4] intel/decoder: Avoid freeing invalid pointer

2018-09-06 Thread Sagar Ghuge
Please ignore this patch. On 09/06/2018 11:33 AM, Sagar Ghuge wrote: > v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin) > > v3: Handle case where genxml is empty (Lionel Landwerlin) > > Signed-off-by: Sagar Ghuge > --- > src/intel/common/gen_decoder.c | 17 -

[Mesa-dev] [PATCH v3 2/4] intel/decoder: Avoid freeing invalid pointer

2018-09-06 Thread Sagar Ghuge
v2: Free ctx.spec if error while reading genxml (Lionel Landwerlin) v3: Handle case where genxml is empty (Lionel Landwerlin) Signed-off-by: Sagar Ghuge --- src/intel/common/gen_decoder.c | 17 - 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/src/intel/common/gen