erreur de segmentation pendant memcpy

Question

J'essaie de faire une fonction qui inverse l'ordre d'une partie d'une chaîne. Je suis nouveau à l'utilisation de pointeurs et pour une raison quelconque, je peux accéder à l'emplacement des caractères de ma chaîne à copier un sous-chaîne, mais je ne peux pas les remettre dans le même endroit ...

Lorsque je tente pour copier la sous-chaîne retournée dans son emplacement d'origine, je reçois

Signal de réception de programme SIGSEGV, Défaut de segmentation. 0x00007ffff7b5dc66 dans ??() from /lib/x86_64-linux-gnu/libc.so.6

Toute aide serait géniale!

Voici ce que j'ai jusqu'à présent:

void reverse(char* line, int start, int end){ 

     char str[end-start]; 

     memcpy (str , line + start , end-start); 

     reverseSubString (str); 

     memcpy (line + start, str , end-start); 

} 

void reverseSubString(char* str){ 

    int i, j, len; 
    char temp; 

    i=j=len=temp=0; 

    len=strlen(str); 

    for (i=0, j=len-1; i<=j; i++, j--) 
    { 
     temp=str[i]; 
     str[i]=str[j]; 
     str[j]=temp; 
    } 
} 

Answer 1

char str[end-start]; 

    memcpy (str , line + start , end-start); 

À moins que le 0-terminaison de la chaîne d'origine est inclus, vous avez un tableau char qui n'est pas 0 TERMINAISON. Donc

len=strlen(str); 

calcule qui-sait-quoi si cela ne plante pas déjà. Puis

for (i=0, j=len-1; i<=j; i++, j--) 
{ 
    temp=str[i]; 
    str[i]=str[j]; 
    str[j]=temp; 
} 

accède à l'extérieur de la mémoire allouée.

Answer 2

reverseSubstring() attend une chaîne C terminée par un NUL, parce qu'il appelle strlen() sur la chaîne passée. Cependant, vous ne NUL à la fin de votre chaîne dans la fonction reverse. Soit le faire, ou mieux encore, passer un argument length:

void reverseSubString(char *str, size_t len) 
{ 
    int i, j; 
    char temp; 

    for (i = 0, j = len - 1; i <= j; i++, j--) { 
     temp = str[i]; 
     str[i] = str[j]; 
     str[j] = temp; 
    } 
} 

void reverse(char *line, int start, int end) 
{ 
    char str[end - start]; 
    memcpy(str, line + start, end - start); 
    reverseSubString(str, end - start); 
    memcpy (line + start, str, end - start); 
}