conseguir "la segmentación falló" error en la implementación de la función strrev

Question

void main() { 
    void strrev(char *); 
    char *p="GOOd"; 
    strrev(p); 
    printf("%s",p); 
} 

void strrev(char *str) { 
    char temp, *end_ptr; 
    if(str == NULL || !(*str)) return; 
    end_ptr = str + strlen(str) - 1; 

    while(end_ptr > str) 
    { 
     temp = *str; 
     *str = *end_ptr; 
     *end_ptr = temp; str++; 
     end_ptr--; 
    } 
} 

estoy recibiendo la segmentación de error no puede cualquiera que me ayude encontrar la manera de solucionar el problema ...

Answer 1

Un problema es que en el siguiente:

char *p = "GOOd"; 

se permite colocar la cadena literal en la memoria de sólo lectura del compilador.

Cualquier intento de modificar la cadena apuntada por p da como resultado un comportamiento indefinido.

trate de cambiar la línea anterior a:

char p[] = "GOOd"; 

no veo nada de malo en la función strrev() sí.

Answer 2

La declaración:

char *p = "GOOd"; 

Define una cadena literal "BUENA", apuntada por un puntero p.

Está intentando modificar este literal de cadena mediante la función strrev que da como resultado Comportamiento no definido (UB) y el bloqueo.

El problema con los literales de cadena es que se almacenan en una ubicación de memoria de solo lectura (Implementación definida) y los programas de usuario no pueden cambiar eso. Si un programa intenta hacerlo, causa un UB.

Por lo tanto, en lugar de utilizar un literal de cadena, debe utilizar una matriz.

Debería utilizar:

char p[] = "GOOd";