Re: [Mesa-dev] [PATCH] radv: fix a potential crash if attachments allocation failed

2017-09-14 Thread Bas Nieuwenhuizen
Reviewed-by: Bas Nieuwenhuizen On Thu, Sep 14, 2017 at 6:47 PM, Samuel Pitoiset wrote: > Also, it's useless to set the error code twice. Though, we > should probably skip the next commands when the command buffer > is considered invalid. > > Signed-off-by: Samuel Pitoiset > --- > src/amd/vulka

[Mesa-dev] [PATCH] radv: fix a potential crash if attachments allocation failed

2017-09-14 Thread Samuel Pitoiset
Also, it's useless to set the error code twice. Though, we should probably skip the next commands when the command buffer is considered invalid. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_cmd_buffer.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/amd/vulka