Vulkan: Fix edge case in DynamicBuffer causing wrong allocations

- on a very specific scenario when the next buffer to allocate was fitting exactly in the allocated size. The condition was wrong and needed to be >= instead of >. Bug: angleproject:2562 Change-Id: Ide36bc3baadafdc1ffa40416b46998dc455caecf Reviewed-on: https://chromium-review.googlesource.com/1075293 Commit-Queue: Luc Ferron <lucferron@chromium.org> Reviewed-by: Geoff Lang <geofflang@chromium.org> Reviewed-by: Jamie Madill <jmadill@chromium.org>

Vulkan: Fix edge case in DynamicBuffer causing wrong allocations
574a6f25 · Luc Ferron · Commit Bot · 22695bf5 · 574a6f25
Commit 574a6f25 authored May 28, 2018 by Luc Ferron Committed by Commit Bot May 30, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

vk_helpers.cpp src/libANGLE/renderer/vulkan/vk_helpers.cpp +1 -1

No files found.
--- a/src/libANGLE/renderer/vulkan/vk_helpers.cpp
+++ b/src/libANGLE/renderer/vulkan/vk_helpers.cpp
@@ -126,7 +126,7 @@ Error DynamicBuffer::allocate(RendererVk *renderer,
    angle::base::CheckedNumeric<size_t> checkedNextWriteOffset = mNextAllocationOffset;
    checkedNextWriteOffset += sizeToAllocate;

-    if (!checkedNextWriteOffset.IsValid() || checkedNextWriteOffset.ValueOrDie() > mSize)
+    if (!checkedNextWriteOffset.IsValid() || checkedNextWriteOffset.ValueOrDie() >= mSize)
    {
        VkDevice device = renderer->getDevice();