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)

partreturn.c​ (1.6 KB) - added by Nickolay Semyonov-Kolchin on Aug 3, 2010 at 5:32:07 AM.
error testcase.

Download all attachments as: .zip

Change History (5)

Changed on Aug 3, 2010 at 5:32:07 AM by Nickolay Semyonov-Kolchin

Attachment: partreturn.c​ added

error testcase.

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
Note: See TracTickets for help on using tickets.