[PATCH 2/4] score: Return a status in _Objects_Set_name()
Sebastian Huber
sebastian.huber at embedded-brains.de
Tue Nov 24 12:52:22 UTC 2020
---
cpukit/include/rtems/score/objectimpl.h | 14 ++++++++------
cpukit/score/src/objectsetname.c | 6 +++---
testsuites/psxtests/psxobj01/init.c | 6 +++---
3 files changed, 14 insertions(+), 12 deletions(-)
diff --git a/cpukit/include/rtems/score/objectimpl.h b/cpukit/include/rtems/score/objectimpl.h
index 32387594f2..a963396d65 100644
--- a/cpukit/include/rtems/score/objectimpl.h
+++ b/cpukit/include/rtems/score/objectimpl.h
@@ -25,6 +25,7 @@
#include <rtems/score/apimutex.h>
#include <rtems/score/assert.h>
#include <rtems/score/isrlock.h>
+#include <rtems/score/status.h>
#include <rtems/score/sysstate.h>
#include <rtems/score/threaddispatch.h>
@@ -463,14 +464,15 @@ size_t _Objects_Name_to_string(
* or up to the first four characters of the string based upon
* whether this object class uses strings for names.
*
- * @param information points to the object information structure
- * @param[out] the_object is the object to operate upon
- * @param name is a pointer to the name to use
+ * @param information points to the object information.
+ * @param[out] the_object is the object to operate upon.
+ * @param name is a pointer to the name to use.
*
- * @retval true The operation succeeded.
- * @retval false The operation failed.
+ * @retval STATUS_SUCCESSFUL The operation succeeded.
+ *
+ * @retval STATUS_NO_MEMORY There was no memory available to duplicate the name.
*/
-bool _Objects_Set_name(
+Status_Control _Objects_Set_name(
const Objects_Information *information,
Objects_Control *the_object,
const char *name
diff --git a/cpukit/score/src/objectsetname.c b/cpukit/score/src/objectsetname.c
index 0808ab87af..4d85332711 100644
--- a/cpukit/score/src/objectsetname.c
+++ b/cpukit/score/src/objectsetname.c
@@ -24,7 +24,7 @@
#include <string.h>
-bool _Objects_Set_name(
+Status_Control _Objects_Set_name(
const Objects_Information *information,
Objects_Control *the_object,
const char *name
@@ -37,7 +37,7 @@ bool _Objects_Set_name(
length = strnlen( name, information->name_length );
dup = _Workspace_String_duplicate( name, length );
if ( dup == NULL ) {
- return false;
+ return STATUS_NO_MEMORY;
}
the_object->name.name_p = dup;
@@ -59,5 +59,5 @@ bool _Objects_Set_name(
_Objects_Build_name( c[ 0 ], c[ 1 ], c[ 2 ], c[ 3 ] );
}
- return true;
+ return STATUS_SUCCESSFUL;
}
diff --git a/testsuites/psxtests/psxobj01/init.c b/testsuites/psxtests/psxobj01/init.c
index ba1e327ea5..fef749b3e9 100644
--- a/testsuites/psxtests/psxobj01/init.c
+++ b/testsuites/psxtests/psxobj01/init.c
@@ -37,7 +37,7 @@ static rtems_task Init(
Objects_Control *the_object;
char name[64];
size_t name_len;
- bool bc;
+ Status_Control status;
TEST_BEGIN();
@@ -70,12 +70,12 @@ static rtems_task Init(
puts( "INIT - _Objects_Set_name fails - out of memory" );
rtems_workspace_greedy_allocate( NULL, 0 );
- bc = _Objects_Set_name(
+ status = _Objects_Set_name(
&Test_Information,
&_Thread_Get_executing()->Object,
name
);
- rtems_test_assert( bc == false );
+ rtems_test_assert( status == STATUS_NO_MEMORY );
TEST_END();
rtems_test_exit(0);
--
2.26.2
More information about the devel
mailing list