Important bugfix for Linux MQ #394
No reviewers
Labels
No Label
API Change
Breaking API Change
bug
build
cosmetics
Documentation
duplicate
feature
help wanted
hotfix
invalid
question
Refactor
Tests
wontfix
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: fsfw/fsfw#394
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "KSat/fsfw:linux/hotfix-mq"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This bug was not an issue on hosted Linux or Rapsberry Pi (at least on the surface), but causes crashes on a Zynq SoC.
A buffer write was exceeded by 4 bytes, causing buffer overflow errors on that system.
The former implementation used the hardcoded value
MAX_DATA_SIZE
.I replaced this with
getMaximumMessageSize()
but this is actually a problem because this will return the size with the header as well.I suspect even a unit test would not have caught this because this was not an obvious issue on host systems. Maybe
valgrind
or a static code analyzer would have caught it..super evil bugfix for Linux MQto Important bugfix for Linux MQBug is also in OSAL for RTEMS!