Changeset 96e9434 in mainline for uspace/app/bdsh/exec.c

Timestamp:

2018-12-29T18:28:21Z (5 years ago)

Author:

Matthieu Riolo <matthieu.riolo@…>

Children:

e70f1ae

Parents:

accdbd8

Message:

correcting find_command() which returns allocated strings

The old version of find_command() relied on a global variable called
'found'. This variable had to be released after every call on
find_command(). The old version did this immediatly after the call.

The code directly implied that this has to be changed with the following
comment:
'FIXME: Just have find_command() return an allocated string'

This commit removes the global variable, changes the implementation
of find_command() to return an allocated string and adds/removes
certain str_dup() calls to match the new implementation

File:

• uspace/app/bdsh/exec.c (modified) (4 diffs)

uspace/app/bdsh/exec.c

@@ -47 +47 @@
 #include "errors.h"
 
-/* FIXME: Just have find_command() return an allocated string */
-static char *found;
-
 static char *find_command(char *);
 static int try_access(const char *);
@@ -76 +73 @@
         size_t i;
 
-        found = (char *)malloc(PATH_MAX);
-
         /* The user has specified a full or relative path, just give it back. */
         if (-1 != try_access(cmd)) {
-                return (char *) cmd;
+                return str_dup(cmd);
         }
 
+        char *found = (char *)malloc(PATH_MAX);
         /* We now have n places to look for the command */
         for (i = 0; search_dir[i] != NULL; i++) {
@@ -91 +87 @@
                 }
         }
+        free(found);
 
         /* We didn't find it, just give it back as-is. */
-        return (char *) cmd;
+        return str_dup(cmd);
 }
 
@@ -107 +104 @@
         FILE *files[3];
 
-        tmp = str_dup(find_command(cmd));
-        free(found);
+        tmp = find_command(cmd);
 
         files[0] = io->stdin;