[PATCH v2] pwdgrp.c: removed assert causing runtime issues

Ryan Long ryan.long at oarcorp.com
Thu Mar 18 17:40:06 UTC 2021


Changed the _Assert_unused_variable_equals macro to just a (void) due to
/etc having already been created by the network stack initialization
or an initial filesystem image.

Updates #4282
---
 cpukit/libcsupport/src/pwdgrp.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/cpukit/libcsupport/src/pwdgrp.c b/cpukit/libcsupport/src/pwdgrp.c
index f4a10f7..edc8aff 100644
--- a/cpukit/libcsupport/src/pwdgrp.c
+++ b/cpukit/libcsupport/src/pwdgrp.c
@@ -36,7 +36,6 @@
 #include <stdint.h>
 
 #include <rtems/seterr.h>
-#include <rtems/score/assert.h>
 
 #include "pwdgrp.h"
 
@@ -63,13 +62,13 @@ static void init_file(const char *name, const char *content)
  */
 static void pwdgrp_init(void)
 {
-  int sc;
-
   /*
    * Do the best to create this directory.
+   *
+   * /etc could be created by the network stack initialization or an initial
+   * filesystem image. Deliberately ignore the return value.
    */
-  sc = mkdir("/etc", S_IRWXU | S_IRGRP | S_IXGRP | S_IROTH | S_IXOTH);
-  _Assert_Unused_variable_equals(sc, 0);
+  (void) mkdir("/etc", S_IRWXU | S_IRGRP | S_IXGRP | S_IROTH | S_IXOTH);
 
   /*
    *  Initialize /etc/passwd
-- 
1.8.3.1



More information about the devel mailing list