Opened on Aug 3, 2010 at 5:32:07 AM
Last modified on Aug 9, 2010 at 6:58:10 AM
#1650 closed defect (wontfix)
rtems_partition_return_buffer can be called several times
Reported by: | Nickolay Semyonov-Kolchin | Owned by: | Joel Sherrill |
---|---|---|---|
Priority: | normal | Milestone: | 4.10 |
Component: | doc | Version: | 4.10 |
Severity: | normal | Keywords: | |
Cc: | sebastian.huber@… | Blocked By: | |
Blocking: |
Description
rtems_partition_return_buffer has no protection for multiple calls:
Sample code:
rtems_partition_get_buffer(part_id, &buf);
sc1 = rtems_partition_return_buffer(part_id, buf);
sc2 = rtems_partition_return_buffer(part_id, buf);
Both sc1 and sc2 are equal to RTEMS_SUCCCESSFUL. Internally partition "memory" member contains two "buf" inside Chain. And "number_of_used_blocks" is equal to
-1.
Proposal: Scan "memory" chain to prevent adding buffers that are already inside chain. I didn't find any standard API for searchin data inside chain.
Attachments (1)
Change History (5)
Changed on Aug 3, 2010 at 5:32:07 AM by Nickolay Semyonov-Kolchin
Attachment: | partreturn.c added |
---|
comment:1 Changed on Aug 3, 2010 at 5:39:32 AM by Sebastian Huber
Resolution: | → wontfix |
---|---|
Status: | new → closed, sebastian.huber@embedded-brains.de |
comment:2 Changed on Aug 7, 2010 at 12:26:25 AM by Joel Sherrill
Component: | cpukit → doc |
---|---|
Milestone: | 4.11 → 4.10 |
comment:3 Changed on Aug 9, 2010 at 6:58:10 AM by Sebastian Huber
Replying to comment:5:
Documentation needs to be updated on branches and head.
Committed the documentation patch also to 4.10 branch. I am not able to patch all branches.
comment:4 Changed on Nov 24, 2014 at 6:56:19 PM by Gedare Bloom
Version: | HEAD → 4.10 |
---|
error testcase.