Erreur de segmentation à l'aide d'execvp dans le shell c

Question

#include <stdio.h> 
#include <sys/types.h> 
#include <sys/wait.h> 
#include <unistd.h> 
#include <stdlib.h> 
#include <string.h> 

void main(){ 
char *cmd; 
pid_t pid; 
while (1) { 
printf("$ "); 
fgets(cmd,1000,stdin); 
if (pid = fork() == -1) { 
exit(1); 
} 
else if (pid == 0){ 
execvp(cmd,&cmd); 
} 
else{ 
int status; 
wait(&status); 
} 
} 
} 

Je crée un shell simple qui exécute des commandes, mais lorsque j'entre la commande à l'invite, je continue d'obtenir une erreur de segmentation. Ceci est la plus simple version qui ne fonctionne que pour les commandes à un argument comme "ls"

Answer 1

Pour votre problème, dans votre code,

fgets(cmd,1000,stdin); 

cmd est non initialisée. Cela ne pointe pas vers une mémoire valide. L'accès à la mémoire invalide appelle undefined behavior.

Vous devez allouer de la mémoire à cmd avant de pouvoir l'utiliser. Alternativement, vous pouvez envisager de créer un tableau cmd, comme char cmd[1000] = {0}; pour éviter d'avoir à allouer de la mémoire vous-même.

Ensuite, execvp(cmd,&cmd); n'est pas tout à fait raison, ce n'est pas ce que vous pensez. Lisez le man page pour une meilleure compréhension. Cela dit, pour un environnement hébergé, void main() devrait au moins être int main(void) pour avoir la conformité standard.