所以我必须实现对执行外部命令的支持(对于C编程语言中的linux)。这就是我到目前为止所拥有的,我使用readline库作为历史函数,但这无关紧要。。。有人能告诉我我做错了什么吗?(我想这就是我所说的"execvp")这是我的代码:
#include<fcntl.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <unistd.h>
#include <readline/readline.h>
#include <readline/history.h>
#define BUFFER_SIZE 256
#define READFILE_SIZE 4096
char ** parseCMD( char * );
void EXEC(char *, char *);
int main(int argc, char ** argv)
{
char myPrompt[]= {'>', '_', ' '};
char *currLine = (char* ) malloc(sizeof(char) * BUFFER_SIZE),
*command = (char* ) malloc(sizeof(char) * BUFFER_SIZE),
*argument = (char* ) malloc(sizeof(char) * BUFFER_SIZE);
currLine = readline(myPrompt);
while((strcmp(currLine, "exit") != 0))
{
command = strtok(currLine, " ");
argument = strtok( NULL, "");
EXEC(command, argument);
currLine = readline(myPrompt);
}
return 0;
}
char ** parseCMD( char * buff )
{
int i = 0, n = strlen(buff), j, count = 0;
char ** CMDargs = (char **) malloc( sizeof( char ) * 100 * 100);
if( buff == NULL )
return NULL;
for(i; i < n; i ++)
{
j = 0;
char * aux = (char *) malloc( sizeof( char ) * 100);
while( buff[i] != ' ' || buff[i] != 't' || buff[i] != 'n')
aux[j++] = buff[i++];
aux[j] = ' ';
CMDargs[count] = strdup( aux );
count++;
//printf("Argument %d is: %s", count - 1, CMDargs[count - 1]);
free(aux);
}
CMDargs[ count ] = NULL;
return CMDargs;
}
void EXEC(char *command, char *argBuffer)
{
pid_t pid;
int status, fd[2], n;
char s[255];
char ** Args;
pipe( fd );
Args = parseCMD( argBuffer );
if ((pid = fork()) < 0)
{
printf("ERROR: forking child process failedn");
exit(1);
}
else if (pid == 0)
{
close(fd[0]);
dup2(fd[1],1);
if (execvp(command, Args) < 0)
{
printf("ERROR: execvp call failedn");
exit(1);
}
close(fd[1]);
}
else
{
close(fd[1]);
while( ( n = read( fd[0], s, 255 ) ) > 0 )
{
s[n] = ' ';
printf("%s",s);
}
while (wait(&status) != pid);
close(fd[0]);
}
}
这里面有太多的错误,问问自己,当我输入空命令时会发生什么(变量命令为NULL,稍后会得到segfault),如果我输入没有参数的命令(变量参数为NULL,您稍后会得到segfault)会发生什么,如果我输入带参数的命令会发生什么?(而函数parseCMD中的循环永远不会终止,最终您将访问一些不应该访问的内容)。也许是时候学习使用调试器了。试着一行一行地逐步执行程序,观察发生了什么。