New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

STM32: HAL smartcard, fix memory corruption in Receive #5505

Merged
merged 1 commit into from Nov 16, 2017

Conversation

Projects
None yet
3 participants
@LMESTM
Contributor

LMESTM commented Nov 15, 2017

This is a follow-up of PR #5303. In order to avoid losing that was proposed in #5303 but that could not be merged as it is we are proposing this new PR. We are taking only the fix part, and leaving behind application related changes.

Proposal: close #5303 and use this PR instead.

Description of fix

Re-casting with tmp the uint8_t* pData pointer to uint16_t* brings a
memory corruption and typically can corrupt the size parameter. This
is fixed with this commit.

STM32 Internal ticket reference : 39116

STM32: HAL smartcard, fix memory corruption in Receive
Re-casting with tmp the uint8_t* pData pointer to uint16_t* brings a
memory corruption and typically can corrupt the size parameter. This
is fixed with this commit.

STM32 Internal ticket reference : 39116

@0xc0170 0xc0170 added the needs: CI label Nov 15, 2017

@0xc0170

This comment has been minimized.

Member

0xc0170 commented Nov 15, 2017

/morph build

@mbed-ci

This comment has been minimized.

mbed-ci commented Nov 15, 2017

Build : SUCCESS

Build number : 523
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/5505/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build

@mbed-ci

This comment has been minimized.

@mbed-ci

This comment has been minimized.

@0xc0170 0xc0170 merged commit dc6dd59 into ARMmbed:master Nov 16, 2017

6 checks passed

AWS-CI uVisor Build & Test Success
Details
ci-morph-build build completed
Details
ci-morph-exporter build completed
Details
ci-morph-test test completed
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment