tree 87adb5aa9a4b065530ef81d3321dacdfca1194d0
parent 97328600dbe369be3e4dfd92dea082f577d35a62
author Archit Gupta <archigup@amazon.com> 1781290513 +0000
committer GitHub <noreply@github.com> 1781290513 +0000
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsFcBAABCAAQBQJqLFYRCRC1aQ7uu5UhlAAARCgQAGs/U9hrSIPUSNrIZsLFu29T
 l+THx3sfw9KWbBebXOxVRb359J23JWa1JurGJ8r6e+IlUPTISw8ar2yOQM0UCU5U
 as7qPOB/rbwrfMY+o+qBuVfFAjEgO/CpiPUQitQI7IAB9qGsHTo+fsuvC40SIK0a
 zI0E9epVKF36c5hMVt+i12t9GGsQrKMBJwVuPpkJFskqvnb/pFI2B6/JkaFmOZbV
 hLzI3RkK/1u+5TisoxYdMQXkMNNQ94n+rnJbH1gpG+2XpShbEJgeElPKyuBIzVPD
 oyC8QEUV5l3HWXO0qTE3Binz76hh3wJ+ZBT4k/eWfz9QRn/gFKYMIVmKIn819/RI
 5/0vaUoNGk2zl7p4eaHJFGpgDPGehPjdrptZrVhc2+gX3J2oFjDnB0aixpRGMg31
 XiSE0p7GsTVab5oKLST4jTgDb6E34VQMQC53aav+AQsZovrFC+qxOV0Dykr+sbko
 5UHwffpecdkw5pPIfRFerIJm6PxkdE0VOrpEOeWpneGGhO3g4SskfZ0WlAgXdxhy
 RqTuCvMVEEzrZu+ORq+6LduqeEHdrcaBMkkcLPAwx9gD7f0wQ/4bpkOMnhfLvGP9
 kn9BXFMZ45zb8ripbqkwKj3+ayVCFQBqIwu/oznHYnOe2wthboVTYJpfvSGW/alw
 NuwQsfTiHpB5jgX5C2kq
 =voaM
 -----END PGP SIGNATURE-----
 

Fix spurious heap_5 assert when allocation search reaches end marker (#1425)

When configENABLE_HEAP_PROTECTOR is 1 and a pvPortMalloc() request cannot
be satisfied by any free block, the free list search advances onto the end
marker (pxEnd) and validates it with heapVALIDATE_BLOCK_POINTER().

pxEnd is located at pucHeapHighAddress and is not part of the usable heap,
so the macro's "( pxBlock ) < pucHeapHighAddress" bounds check fails and the
configASSERT() fires on what is a normal out-of-memory condition. The code
should instead fall through and call the malloc failed hook.

Exclude the end marker from validation in the search loop so an allocation
that cannot be satisfied returns NULL (and invokes vApplicationMallocFailedHook
when configUSE_MALLOC_FAILED_HOOK is 1) without asserting.