From 2f2d85e30d941f287f2771d167d3be5cc814dae2 Mon Sep 17 00:00:00 2001 From: Woochan Lee Date: Thu, 22 May 2025 19:52:17 +0900 Subject: [PATCH] Revert "(UsdLoader) Fix coverity issue - uint32 can't have less than 0 value" This reverts commit 37ace7d9b491cadaeeabf472b16b80922e37782d. --- dali-usd-loader/internal/usd-loader-impl.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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(x) >= rawUVs.size())) + if(DALI_UNLIKELY(x < 0 || static_cast(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. -- 2.34.1