Merge pull request #137 from neboskreb/bugfix/136-queue-change-cast

feilipu · web-flow · commit a4cdf789d7cc · 2024-09-30T08:49:50.000+10:00
diff --git a/src/queue.c b/src/queue.c
@@ -506,7 +506,7 @@ BaseType_t xQueueGenericReset( QueueHandle_t xQueue,
             /* Check for multiplication overflow. */
             ( ( SIZE_MAX / uxQueueLength ) >= uxItemSize ) &&
             /* Check for addition overflow. */
-            ( ( UBaseType_t ) ( SIZE_MAX - sizeof( Queue_t ) ) >= ( uxQueueLength * uxItemSize ) ) )
+            ( ( SIZE_MAX - sizeof( Queue_t ) ) >= ( size_t ) ( uxQueueLength * uxItemSize ) ) )
         {
             /* Allocate enough space to hold the maximum number of items that
              * can be in the queue at any time.  It is valid for uxItemSize to be

Original file line number	Diff line number	Diff line change
`@@ -506,7 +506,7 @@ BaseType_t xQueueGenericReset( QueueHandle_t xQueue,`
`506`	`506`	`/* Check for multiplication overflow. */`
`507`	`507`	`( ( SIZE_MAX / uxQueueLength ) >= uxItemSize ) &&`
`508`	`508`	`/* Check for addition overflow. */`
`509`		`- ( ( UBaseType_t ) ( SIZE_MAX - sizeof( Queue_t ) ) >= ( uxQueueLength * uxItemSize ) ) )`
	`509`	`+ ( ( SIZE_MAX - sizeof( Queue_t ) ) >= ( size_t ) ( uxQueueLength * uxItemSize ) ) )`
`510`	`510`	`{`
`511`	`511`	`/* Allocate enough space to hold the maximum number of items that`
`512`	`512`	`* can be in the queue at any time. It is valid for uxItemSize to be`