From: Alex Smith <alex@alex-smith.me.uk>
Date: Fri, 23 Sep 2016 11:18:51 +0000 (+0100)
Subject: layers: Fix VerifyCopyUpdateContents on combined image sampler descriptors
X-Git-Tag: accepted/tizen/3.0.m2/mobile/20170105.023708~126
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dd75b52c0bd5aadb682f157107bd59c2715d1d1b;p=platform%2Fupstream%2FVulkan-LoaderAndValidationLayers.git

layers: Fix VerifyCopyUpdateContents on combined image sampler descriptors

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
---

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