strcat fonctionnant paranormalement

Question

J'essaie de passer des arguments de ligne de commande que je concatène ensuite correctement pour générer des commandes shell afin que je puisse les exécuter en utilisant system() (je sais que ce n'est pas conseillé et il y a de meilleurs moyens pour le faire de cette façon seulement). Mais il y a un problème dans la concaténation des chaînes que je passe Voici le code (j'ai tout imprimé à chaque étape pour avoir une compréhension claire et non je n'ai pas encore écrit les appels system(), d'abord j'ai besoin de trier concaténation dehors):

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
int main(int argc, char *argv[]) { 
    char* path=argv[1]; 
    char* oldName=argv[2]; 
    char* newName=argv[3]; 
    char* command1="cd "; 
    char* command2="ren "; 
     printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 
    strcat(command1,path); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 
    strcat(oldName," "); 
    strcat(oldname,newName); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 
    strcat(command2,oldName); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 

    return 0; 
} 

Cependant, après la concaténation de la commande 1 au chemin, tout est corrompu.

Answer 1

Vous devez allouer de l'espace en utilisant un tampon de caractères pour empêcher le strcat de se briser, comme ceci:

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
int main(int argc, char *argv[]) { 
    char* path=argv[1]; 
    char oldName[256]; 
    char* newName=argv[3]; 
    char command1[256]; 
    char command2[256]; 
    strcpy(command1, "cd "); 
    strcpy(command2, "ren "); 
    strcpy(oldName, argv[2]); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 
    strcat(command1,path); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 
    strcat(oldName," "); 
    strcat(oldName,newName); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 
    strcat(command2,oldName); 
    printf("\n\n%s\n%s\n%s\n%s\n%s\n",command1,command2,path,oldName,newName); 

    return 0; 
} 

Answer 2

strcat œuvres en copiant octets de la chaîne source à la fin de la chaîne de destination. Le problème est que vos chaînes de destination sont:

1. chaînes constantes (et ne peuvent pas être en mémoire inscriptible)
2. Pas assez longtemps pour tenir le résultat entier

Vous devriez probablement créer un char buffer comme char buffer[1024] pour contenir les commandes et utiliser snprintf pour formater les commandes dans le tampon.

Answer 3

strcat s'attend à ce que la destination soit suffisamment grande pour contenir le résultat. Pour quote:

pointeur vers le tableau de destination, qui devrait contenir une chaîne C, et être assez grand pour contenir la chaîne résultante concaténer.