сегментации при освобождении указателя при движении задним ходом строки в C

Question

int main(int argc, char **argv){ 
    char *str = argv[1]; 
    size_t len = strlen(str); 
    char *dst = calloc(len+1, sizeof(char)); 
    int i; 
    for(i=len-1; i>=0; i--){ 
     memcpy(dst, str+i, 1); 
     dst++; 
    } 
    *(++dst) = '\0'; 
    printf("%s\n", dst); 
    free(dst); 
    return 0; 
} 

проверки ошибок опущено для получения дополнительной читаемости
Я не понимаю, почему я получил Segfault с этим сообщением: «GLibC свободный(): недопустимый указатель:»
Я пытаюсь написать небольшую программу, которая обращает строку с помощью указателей, но:
1) последний printf ничего не печатает;
2) я получил Segfault

Answer 1

проблема заключается в том, что вы меняете указатель в вычислениях.
попробовать использовать temp или original для этого типа:

int main(int argc, char **argv){ 
    char *str = argv[1]; 
    size_t len = strlen(str); 
    char *dst = calloc(len+1, sizeof(char)); 
    char *origDst = dst; 
    int i; 
    for(i=len-1; i>=0; i--){ 
     memcpy(dst, str+i, 1); 
     dst++; 
    } 
    *(dst) = '\0'; 
    printf("%s\n", origDst); 
    free(origDst); 
    return 0; 
} 

Answer 2

Вы:

char *dst = calloc(len+1, sizeof(char)); 
    /* ... */ 
    *(++dst) = '\0'; 
    /* ... */ 
    free(dst); 

Вы должны free указатель, который был выделен на malloc/calloc. Здесь вы изменили его. Сделайте копию, чтобы вы могли free после.

Answer 3

Есть 2 точки, которые необходимо учитывать. Во-первых, сделать копию исходного указателя, выделенную calloc, как уже сообщалось ouah. Другим моментом является устранение приращения указателя при сохранении NULL, то есть *(++dst) = '\0'; должно быть *dst = '\0';. С этими двумя изменениями я могу запускать вашу программу без каких-либо проблем.