From 788f8cfd7693da0b26a75c6758519d3f514ef497 Mon Sep 17 00:00:00 2001 From: chinglee-iot <61685396+chinglee-iot@users.noreply.github.com> Date: Mon, 26 Jun 2023 15:44:33 +0800 Subject: [PATCH] Update static stream buffer size check (#693) * Use volatile size instead of sizeof directly to prevent always true/false warning --- stream_buffer.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/stream_buffer.c b/stream_buffer.c index cbc0939b3..2306b5d9b 100644 --- a/stream_buffer.c +++ b/stream_buffer.c @@ -436,11 +436,13 @@ static void prvInitialiseNewStreamBuffer( StreamBuffer_t * const pxStreamBuffer, configASSERT( xBufferSizeBytes > sbBYTES_TO_STORE_MESSAGE_LENGTH ); #if ( configASSERT_DEFINED == 1 ) - + { /* Sanity check that the size of the structure used to declare a * variable of type StaticStreamBuffer_t equals the size of the real * message buffer structure. */ - configASSERT( sizeof( StaticStreamBuffer_t ) == sizeof( StreamBuffer_t ) ); + volatile size_t xSize = sizeof( StaticStreamBuffer_t ); + configASSERT( xSize == sizeof( StreamBuffer_t ) ); + } /*lint !e529 xSize is referenced is configASSERT() is defined. */ #endif /* configASSERT_DEFINED */ if( ( pucStreamBufferStorageArea != NULL ) && ( pxStaticStreamBuffer != NULL ) )