Skip to content

Commit df62aa7

Browse files
committed
fix: incorrect uint64_t cast in bitmask limit validation
The LIMIT_FORMAT_BITMASK validation uses deUint32 for checking the limit value but incorrectly casts to deUint64 when logging the actual value. This causes incorrect values to be displayed in the logs. This bug was originally introduced in c82c0fa (2016) and persisted through the de* type removal in 1a09796 (2023).
1 parent 1a0ae3b commit df62aa7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

external/vulkancts/modules/vulkan/api/vktApiFeatureInfo.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -542,7 +542,7 @@ bool validateFeatureLimits(VkPhysicalDeviceProperties *properties, VkPhysicalDev
542542
{
543543
log << TestLog::Message << "limit validation failed, " << featureLimitTable[ndx].name
544544
<< " not valid-limit type bitmask actual is "
545-
<< *((uint64_t *)((uint8_t *)limits + featureLimitTable[ndx].offset)) << TestLog::EndMessage;
545+
<< *((uint32_t *)((uint8_t *)limits + featureLimitTable[ndx].offset)) << TestLog::EndMessage;
546546
limitsOk = false;
547547
}
548548
}

0 commit comments

Comments
 (0)