search

FreeRTOS / FreeRTOS-Kernel

FreeRTOS kernel files only, submoduled into https://github.com/FreeRTOS/FreeRTOS and various other repos.
https://www.FreeRTOS.org
MIT License
2.52k stars 1.05k forks source link

Remove PRIVILEGED_FUNCTION from function definition in xMessageBufferNextLengthBytes #989

Closed kar-rahul-aws closed 5 months ago

kar-rahul-aws commented 5 months ago

Description

The PRIVILEGED_FUNCTION attribute should not be added during function definition, but during the function declaration.

Test Steps

MPU soak test for Message buffer on STM32H743ZI

Before applying the patch

Compilation fails with the error

Error[Pe065]: expected a ";" after __attribute__ in the definition for PRIVILEGED_FUNCTION

After applying the patch

Compilation is successful.

Checklist:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

sonarcloud[bot] commented 5 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8cfa715) 93.53% compared to head (2fabdea) 93.53%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #989 +/- ## ======================================= Coverage 93.53% 93.53% ======================================= Files 6 6 Lines 3200 3200 Branches 889 889 ======================================= Hits 2993 2993 Misses 92 92 Partials 115 115 ``` | [Flag](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/989/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/FreeRTOS/FreeRTOS-Kernel/pull/989/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS) | `93.53% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FreeRTOS#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.