Changeset 06599a1 in mainline for uspace/app/sysctl/main.c

Timestamp:

2020-03-06T19:13:55Z (5 years ago)

Author:

Matthieu Riolo <matthieu.riolo@…>

Children:

3529f148

Parents:

22d990c

git-author:

Matthieu Riolo <matthieu.riolo@…> (2020-02-05 21:48:41)

git-committer:

Matthieu Riolo <matthieu.riolo@…> (2020-03-06 19:13:55)

Message:

Correcting sysctl list-units

One failure was a cast in lib/sysman/src/ctl.c which wrongly used
an enum like a pointer of the type sysarg_t. This resulted into a
memory allocation failure.

The second bug was in the loop of sysctl. For iterating over all
found units the difference was created of two pointers and then
compared to an unsigned int which was down casted to an int.
Unnecessary complicated and a source for casting problems.
It has been replaced with a simple for-loop and an array access

File:

• uspace/app/sysctl/main.c (modified) (1 diff)

uspace/app/sysctl/main.c

@@ -76 +76 @@
         }
 
-        for (unit_handle_t *it = units; it - units < (int)unit_cnt; ++it) {
+        for (size_t i = 0; i < unit_cnt; i++) {
+                unit_handle_t handle = units[i];
                 char name[NAME_BUFFER];
                 unit_state_t state;
 
-                rc = sysman_unit_get_name(*it, name, NAME_BUFFER);
+                rc = sysman_unit_get_name(handle, name, NAME_BUFFER);
                 if (rc != EOK)
                         goto fail;
 
-                rc = sysman_unit_get_state(*it, &state);
+                rc = sysman_unit_get_state(handle, &state);
                 if (rc != EOK)
                         goto fail;