I've found a way to call unix external commands without arguments(ex. "ls", "pwd"). It goes like that:
//Child process
char cwd[1024];
getcwd(cwd, sizeof(cwd));
char *argv[] = {*args, NULL}//(ex.) {"ls", NULL}
char *env[] = {cwd, NULL};
//concat():method that connects 2 strings
char *command_source = concat("/bin/", *args);
execve(command_source, argv, env);
return 0;
I'm trying to convert this code in order to accept external commands with arguments like "ls -l"
Assuming that you know the number of arguments in args and that it's argcs:
...
char **argv = calloc(sizeof(char*), argcs+1);
for (int i=0; i<argcs; i++)
argv[i]=args[i];
argv[argcs]=NULL;
...
If not, you can easily determine argcs by iterating through the array searching for the ending NULL.
You can also make a pipeline; look at how the commands are built up and see the structure, they are arrays that end with 0 and the quotes are stripped:
/* who | awk '{print $1}' | sort | uniq -c | sort -n */
/*static char *cmd0[] = {"who", 0};
static char *cmd1[] = {"awk", "{print $1}", 0};
static char *cmd2[] = {"sort", 0};
static char *cmd3[] = {"uniq", "-c", 0};
static char *cmd4[] = {"sort", "-n", 0};*/
Here are some utility function when you create a pipeline with arguments. They are well tested and bug-free.
pipeline.c
#define _XOPEN_SOURCE 500
/* One way to create a pipeline of N processes */
#ifndef STDERR_H_INCLUDED
#define STDERR_H_INCLUDED
static void err_sysexit(char const *fmt, ...);
static void err_syswarn(char const *fmt, ...);
#endif /* STDERR_H_INCLUDED */
/* pipeline.c */
#include <assert.h>
#include <stdio.h>
#include <string.h>
#include <sys/wait.h>
#include <unistd.h>
#include <stdlib.h>
#include "openshell.h"
#include <errno.h>
/* exec_nth_command() and exec_pipe_command() are mutually recursive */
static void exec_pipe_command(int ncmds, char ***cmds, Pipe output);
/* With the standard output plumbing sorted, execute Nth command */
static void exec_nth_command(int ncmds, char ***cmds) {
assert(ncmds >= 1);
if (ncmds > 1) {
pid_t pid;
Pipe input;
if (pipe(input) != 0)
err_sysexit("Failed to create pipe");
if ((pid = fork()) < 0)
err_sysexit("Failed to fork");
if (pid == 0) {
/* Child */
exec_pipe_command(ncmds - 1, cmds, input);
}
/* Fix standard input to read end of pipe */
dup2(input[0], 0);
close(input[0]);
close(input[1]);
}
execvp(cmds[ncmds - 1][0], cmds[ncmds - 1]);
err_sysexit("Failed to exec %s", cmds[ncmds - 1][0]);
/*NOTREACHED*/
}
/* Given pipe, plumb it to standard output, then execute Nth command */
static void exec_pipe_command(int ncmds, char ***cmds, Pipe output) {
assert(ncmds >= 1);
/* Fix stdout to write end of pipe */
dup2(output[1], 1);
close(output[0]);
close(output[1]);
exec_nth_command(ncmds, cmds);
}
/* who | awk '{print $1}' | sort | uniq -c | sort -n */
/*static char *cmd0[] = {"who", 0};
static char *cmd1[] = {"awk", "{print $1}", 0};
static char *cmd2[] = {"sort", 0};
static char *cmd3[] = {"uniq", "-c", 0};
static char *cmd4[] = {"sort", "-n", 0};*/
/*static char **cmds[] = {cmd0, cmd1, cmd2, cmd3, cmd4};*/
/*static int ncmds = sizeof(cmds) / sizeof(cmds[0]);*/
/* Execute the N commands in the pipeline */
void exec_pipeline(int ncmds, char ***cmds) {
assert(ncmds >= 1);
pid_t pid;
if ((pid = fork()) < 0)
err_syswarn("Failed to fork");
if (pid != 0)
return;
exec_nth_command(ncmds, cmds);
}
#include <stdarg.h>
static const char *arg0 = "<undefined>";
static void err_vsyswarn(char const *fmt, va_list args) {
int errnum = errno;
fprintf(stderr, "%s:%d: ", arg0, (int) getpid());
vfprintf(stderr, fmt, args);
if (errnum != 0)
fprintf(stderr, " (%d: %s)", errnum, strerror(errnum));
putc('\n', stderr);
}
static void err_syswarn(char const *fmt, ...) {
va_list args;
va_start(args, fmt);
err_vsyswarn(fmt, args);
va_end(args);
}
static void err_sysexit(char const *fmt, ...) {
va_list args;
va_start(args, fmt);
err_vsyswarn(fmt, args);
va_end(args);
exit(1);
}
The code comes from a prior shell project from an earlier question where a bounty was awarded for the above code. There is an extensive answer already with details — How to fix these errors in my code. There's also the earlier answer to C minishell: adding pipelines which contains most of the quoted code.
You can also read pipe(2), pipe(7) and Advanced Linux Programming
