Fix bounds check for signalIdx

mikex86 · web-flow · commit ea8cd3083201 · 2025-05-18T07:41:37.000+02:00
diff --git a/driverapi/src/cmdqueue.cpp b/driverapi/src/cmdqueue.cpp
@@ -241,7 +241,7 @@ libreCudaStatus_t NvCommandQueue::releaseSignal(NvSignal *signal) {
     NvU32 signalIdx = signal - signalPool;
 
     // ensure pointer was within signal pool and thus valid
-    if (signalIdx > MAX_SIGNAL_POOL_SIZE) {
+    if (signalIdx >= MAX_SIGNAL_POOL_SIZE) {
         LIBRECUDA_FAIL(LIBRECUDA_ERROR_INVALID_VALUE);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -241,7 +241,7 @@ libreCudaStatus_t NvCommandQueue::releaseSignal(NvSignal *signal) {`
`241`	`241`	`NvU32 signalIdx = signal - signalPool;`
`242`	`242`
`243`	`243`	`// ensure pointer was within signal pool and thus valid`
`244`		`- if (signalIdx > MAX_SIGNAL_POOL_SIZE) {`
	`244`	`+ if (signalIdx >= MAX_SIGNAL_POOL_SIZE) {`
`245`	`245`	`LIBRECUDA_FAIL(LIBRECUDA_ERROR_INVALID_VALUE);`
`246`	`246`	`}`
`247`	`247`