Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl>
On Sat, Feb 10, 2018 at 5:33 PM, Eric Anholt <e...@anholt.net> wrote: > The compiler doesn't figure out that we only get result == VK_SUCCESS if > set got initialized. > > Cc: Dave Airlie <airl...@redhat.com> > --- > src/amd/vulkan/radv_descriptor_set.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/amd/vulkan/radv_descriptor_set.c > b/src/amd/vulkan/radv_descriptor_set.c > index bbabfa9b2755..056a5644dfa0 100644 > --- a/src/amd/vulkan/radv_descriptor_set.c > +++ b/src/amd/vulkan/radv_descriptor_set.c > @@ -518,7 +518,7 @@ VkResult radv_AllocateDescriptorSets( > > VkResult result = VK_SUCCESS; > uint32_t i; > - struct radv_descriptor_set *set; > + struct radv_descriptor_set *set = NULL; > > /* allocate a set of buffers for each shader to contain descriptors */ > for (i = 0; i < pAllocateInfo->descriptorSetCount; i++) { > -- > 2.15.0 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev