summaryrefslogtreecommitdiffstats
path: root/cpukit/rtems/src/partreturnbuffer.c
diff options
context:
space:
mode:
authorSebastian Huber <sebastian.huber@embedded-brains.de>2021-08-10 12:42:11 +0200
committerSebastian Huber <sebastian.huber@embedded-brains.de>2021-08-12 10:09:04 +0200
commit9399e12c217343545bf412a01629570377536b39 (patch)
treee8a7f068a355bd18d15e745eec7b3c1611ae2f57 /cpukit/rtems/src/partreturnbuffer.c
parentbsps/arm: More robust SMP start (diff)
downloadrtems-9399e12c217343545bf412a01629570377536b39.tar.bz2
rtems: Fix rtems_partition_return_buffer()
The rtems_partition_return_buffer() wrongly accepted which were exactly at the buffer area end. Use the buffer area limit address for the range checking. Close #4490.
Diffstat (limited to '')
-rw-r--r--cpukit/rtems/src/partreturnbuffer.c17
1 files changed, 10 insertions, 7 deletions
diff --git a/cpukit/rtems/src/partreturnbuffer.c b/cpukit/rtems/src/partreturnbuffer.c
index f5ab7d85f9..68302f1163 100644
--- a/cpukit/rtems/src/partreturnbuffer.c
+++ b/cpukit/rtems/src/partreturnbuffer.c
@@ -33,7 +33,7 @@ static bool _Partition_Is_address_on_buffer_boundary(
offset = _Addresses_Subtract(
the_buffer,
- the_partition->starting_address
+ the_partition->base_address
);
return ( offset % the_partition->buffer_size ) == 0;
@@ -44,14 +44,17 @@ static bool _Partition_Is_address_a_buffer_begin(
const void *the_buffer
)
{
- void *starting;
- void *ending;
+ const void *base;
+ const void *limit;
- starting = the_partition->starting_address;
- ending = _Addresses_Add_offset( starting, the_partition->length );
+ base = the_partition->base_address;
+ limit = the_partition->limit_address;
- return _Addresses_Is_in_range( the_buffer, starting, ending )
- && _Partition_Is_address_on_buffer_boundary( the_partition, the_buffer );
+ if ( !_Addresses_Is_in_range( the_buffer, base, limit ) ) {
+ return false;
+ }
+
+ return _Partition_Is_address_on_buffer_boundary( the_partition, the_buffer );
}
static void _Partition_Free_buffer(