layers: Fix VerifyCopyUpdateContents on combined image sampler descriptors
authorAlex Smith <alex@alex-smith.me.uk>
Fri, 23 Sep 2016 11:18:51 +0000 (12:18 +0100)
committerTobin Ehlis <tobine@google.com>
Fri, 23 Sep 2016 16:26:32 +0000 (10:26 -0600)
A missing break at the end of the ImageSampler case results in falling
through to the Image case, leading to spurious validation errors such as:

  vkUpdateDescriptorsSets() failed copy update from Descriptor Set 0x108
  to Descriptor Set 0x137 with error: Attempted copy update to image
  descriptor failed due to: Invalid VkImageView: 0x100000000

layers/descriptor_sets.cpp

index d7ca025..d08a110 100644 (file)
@@ -1359,6 +1359,7 @@ bool cvdescriptorset::DescriptorSet::VerifyCopyUpdateContents(const VkCopyDescri
                 return false;
             }
         }
+        break;
     }
     case Image: {
         for (uint32_t di = 0; di < update->descriptorCount; ++di) {