Merge pull request #2857 from malaterre/master
authormonojenkins <jo.shields+jenkins@xamarin.com>
Fri, 8 Apr 2016 16:40:17 +0000 (17:40 +0100)
committermonojenkins <jo.shields+jenkins@xamarin.com>
Fri, 8 Apr 2016 16:40:17 +0000 (17:40 +0100)
Remove an undefined behavior

Found using gcc -fsanitize=bounds

mono/utils/lock-free-queue.c

index 464419960de18f73df442a23632400e78c5799ad..849f7046a3a51c53e5ef3876a51496cc6bc2d9ac 100644 (file)
@@ -183,7 +183,7 @@ get_dummy (MonoLockFreeQueue *q)
 static gboolean
 is_dummy (MonoLockFreeQueue *q, MonoLockFreeQueueNode *n)
 {
-       return n >= &q->dummies [0].node && n < &q->dummies [MONO_LOCK_FREE_QUEUE_NUM_DUMMIES].node;
+       return n >= &q->dummies [0].node && n <= &q->dummies [MONO_LOCK_FREE_QUEUE_NUM_DUMMIES-1].node;
 }
 
 static gboolean