From: Woochan Lee <wc0917.lee@samsung.com>
Date: Thu, 22 May 2025 10:52:17 +0000 (+0900)
Subject: Revert "(UsdLoader) Fix coverity issue - uint32 can't have less than 0 value"
X-Git-Tag: dali_2.4.20~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2f2d85e30d941f287f2771d167d3be5cc814dae2;p=platform%2Fcore%2Fuifw%2Fdali-toolkit.git

Revert "(UsdLoader) Fix coverity issue - uint32 can't have less than 0 value"

This reverts commit 37ace7d9b491cadaeeabf472b16b80922e37782d.
---

diff --git a/dali-usd-loader/internal/usd-loader-impl.cpp b/dali-usd-loader/internal/usd-loader-impl.cpp
index b4d3712b0e..0f60138a35 100644
--- a/dali-usd-loader/internal/usd-loader-impl.cpp
+++ b/dali-usd-loader/internal/usd-loader-impl.cpp
@@ -696,7 +696,7 @@ void UsdLoaderImpl::Impl::ProcessMeshTexcoords(MeshDefinition& meshDefinition, s
           // Handle vertex-based UVs
           for(auto x : subIndexArray)
           {
-            if(DALI_UNLIKELY(static_cast<size_t>(x) >= rawUVs.size()))
+            if(DALI_UNLIKELY(x < 0 || static_cast<size_t>(x) >= rawUVs.size()))
             {
               // This should never happen. The USD spec and the “indexed” primvar APIs guarantee that
               // you will never have an index that lies outside the authored-values array.