ash: make tryexec(cmd) parameter const char
authorDenys Vlasenko <vda.linux@googlemail.com>
Fri, 28 Jul 2017 23:20:53 +0000 (01:20 +0200)
committerDenys Vlasenko <vda.linux@googlemail.com>
Fri, 28 Jul 2017 23:20:53 +0000 (01:20 +0200)
Fewer casts this way.

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
shell/ash.c

index f9c78ee789480bb2f26de2e5d4454fce99db0c4a..0de81b325ca027325fba01d2cd4f292d2d315044 100644 (file)
@@ -7687,7 +7687,7 @@ static int builtinloc = -1;     /* index in path of %builtin, or -1 */
 
 
 static void
-tryexec(IF_FEATURE_SH_STANDALONE(int applet_no,) char *cmd, char **argv, char **envp)
+tryexec(IF_FEATURE_SH_STANDALONE(int applet_no,) const char *cmd, char **argv, char **envp)
 {
 #if ENABLE_FEATURE_SH_STANDALONE
        if (applet_no >= 0) {
@@ -7713,7 +7713,7 @@ tryexec(IF_FEATURE_SH_STANDALONE(int applet_no,) char *cmd, char **argv, char **
 #else
        execve(cmd, argv, envp);
 #endif
-       if (cmd != (char*) bb_busybox_exec_path && errno == ENOEXEC) {
+       if (cmd != bb_busybox_exec_path && errno == ENOEXEC) {
                /* Run "cmd" as a shell script:
                 * http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html
                 * "If the execve() function fails with ENOEXEC, the shell
@@ -7730,8 +7730,8 @@ tryexec(IF_FEATURE_SH_STANDALONE(int applet_no,) char *cmd, char **argv, char **
                 * message and exit code 126. For one, this prevents attempts
                 * to interpret foreign ELF binaries as shell scripts.
                 */
-               argv[0] = cmd;
-               cmd = (char*) bb_busybox_exec_path;
+               argv[0] = (char*) cmd;
+               cmd = bb_busybox_exec_path;
                /* NB: this is only possible because all callers of shellexec()
                 * ensure that the argv[-1] slot exists!
                 */