proc_portLookup: fix len check - Githubissues

phoenix-rtos / phoenix-rtos-kernel

Phoenix-RTOS microkernel repository

http://phoenix-rtos.com

BSD 3-Clause "New" or "Revised" License

112 stars 33 forks source link

proc_portLookup: fix len check #583

Closed lukileczo closed 3 weeks ago

lukileczo commented 3 weeks ago

If server returned off by 1 value, it wasn't detected and resulted in memcpy with size -1.

JIRA: RTOS-897

Description

Motivation and Context

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (refactoring, style fixes, git/CI config, submodule management, no code logic changes)

How Has This Been Tested?

[x] Already covered by automatic testing.
[ ] New test added: (add PR link here).
[ ] Tested by hand on: (list targets here).

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing linter checks and tests passed.
[x] My changes generate no new compilation warnings for any of the targets.

Special treatment

[ ] This PR needs additional PRs to work (list the PRs, preferably in merge-order).
[ ] I will merge this PR by myself when appropriate.

github-actions[bot] commented 3 weeks ago

Unit Test Results

7 700 tests ±0 6 985 :white_check_mark: ±0 37m 14s :stopwatch: - 1m 51s 436 suites ±0 715 :zzz: ±0 1 files ±0 0 :x: ±0

Results for commit c158d1ab. ± Comparison against base commit fa2cb465.