RIOT-OS/RIOT
RIOT-OS/RIOT
1
0
Fork 0
mirror of https://github.com/RIOT-OS/RIOT.git synced 2024-12-29 04:50:03 +01:00
Code Releases Activity

tests/sys/malloc: fix test with LLVM

Browse Source
This commit is contained in:
Marian Buschsieweke 2023-05-22 12:53:10 +02:00 committed by Marian Buschsieweke
parent 38eb52cbbb
commit 624e9c664a
No known key found for this signature in database
GPG Key ID: CB8E3238CE715A94
1 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
tests/sys/malloc/main.c
View File

@ -109,10 +109,8 @@ static void free_memory(struct node *head)
printf("Free count: %"PRIu32"\n", freed);
}
int main(void)
static void check_calloc_returns_null_on_overflow(void)
{
uint32_t allocations = 0;
/* modern compilers warn about nonsense calls to calloc, but this is exactly what we want to
* test */
#pragma GCC diagnostic push
@ -121,8 +119,24 @@ int main(void)
#endif
/* test if an overflow is correctly detected by calloc(): the size below overflows by 1 byte */
/* cppcheck-suppress leakReturnValNotUsed; (should return NULL, so nothing to free anyway) */
expect(NULL == calloc(SIZE_MAX / 16 + 1, 16));
size_t nmemb = SIZE_MAX / 16 + 1;
size_t size = 16;
void *p = calloc(nmemb, size);
/* When clang detects that the memory allocated is not actually used, it
* will optimize out the call to `calloc()` and just assume that the
* allocation succeeded. It then optimized out the test `NULL == p` and
* assumes it to always be false. We just print the address to prevent
* that from happening */
printf("calloc(%zu, %zu) = %p\n", nmemb, size, p);
expect(NULL == p);
#pragma GCC diagnostic pop
}
int main(void)
{
uint32_t allocations = 0;
check_calloc_returns_null_on_overflow();
printf("CHUNK_SIZE: %"PRIu32"\n", (uint32_t)CHUNK_SIZE);
printf("NUMBER_OF_TESTS: %d\n", NUMBER_OF_TESTS);