STM32H legacy driver: change parameter to pucGetRXBuffer() #1285
+4
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
STM32H legacy driver: change parameter to pucGetRXBuffer()
This issue was reported on the FreeRTOS forum by userhttps://github.com/kzorer.
This is the essential change:
/* Subtracted 'ipBUFFER_PADDING', which is the size of the meta data. */ pucBuffer = pucGetRXBuffer( ETH_RX_BUF_SIZE - ipBUFFER_PADDING ); configASSERT( pucBuffer != NULL );This was a result of an earlier patch that repair a vulnerability.
Test Steps
Create an application that connects to the Ethernet and try anything, send it a an ICMP ping message and it will fail. It went wrong in this test:
Checklist:
Related Issue
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.