fsl_enet.c memory deallocation bug

kevin2024 — Thu, 01 Aug 2024 17:47:03 GMT

Hello,

I couldn't see anywhere else to post a bug fix, so I will post here.

I have isolated a memory deallocation bug in the SDK ethernet driver code. I am using with SDK version 2.11.0. The bug is located in the ENET_GetRxFrame() function. Briefly, the incorrect code supplies the address of the pointer to the memory to be deallocated rather than the memory to be deallocated. The diff to fix the bug is as follows:

--------------------------- drivers/fsl_enet.c --------------------------- index cd30cab..feedd81 100644 @@ -2537,7 +2537,7 @@ status_t ENET_GetRxFrame(ENET_Type *base, enet_handle_t *handle, enet_rx_frame_s /* Free the incomplete frame buffers. */ while (index-- != 0U) { - handle->rxBuffFree(base, &rxFrame->rxBuffArray[index].buffer, handle->userData, ringId); + handle->rxBuffFree(base, rxFrame->rxBuffArray[index].buffer, handle->userData, ringId); } /* Update left buffers as ready for next coming frame */

Re: fsl_enet.c memory deallocation bug

_Leo_ — Mon, 05 Aug 2024 17:24:37 GMT

Hi,

Thank you so much for your interest in our products and for using our community.

Very good observation, this issue had not been noticed. I will report it to the corresponding department so that it can be corrected in the next review. I appreciate your feedback as it helps us improve our products.

Have a nice day!

topic fsl_enet.c memory deallocation bug in Kinetis Microcontrollers

fsl_enet.c memory deallocation bug

Re: fsl_enet.c memory deallocation bug