diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2021-08-10 12:42:11 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2021-08-12 10:09:04 +0200 |
commit | 9399e12c217343545bf412a01629570377536b39 (patch) | |
tree | e8a7f068a355bd18d15e745eec7b3c1611ae2f57 /cpukit/rtems/src/partreturnbuffer.c | |
parent | bsps/arm: More robust SMP start (diff) | |
download | rtems-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 'cpukit/rtems/src/partreturnbuffer.c')
-rw-r--r-- | cpukit/rtems/src/partreturnbuffer.c | 17 |
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( |