ошибка сегментации после memcpy и бесплатно

Question

Я новый программист на C. Я пытаюсь удалить один элемент из статического массива и скопировать остальное на динамический массив. Программа работала нормально, но после free() я получаю ошибку сегментации. Я думаю, что указатель temp указывает на некоторую память в стеке, поэтому я получаю ошибку сегментации (потому что я пытаюсь освободить что-то, что не создано malloc). Но проблема memcpy должна копироваться из стека в кучу, так что temp остается указывать на кучу. Вот код:

#include <stdlib.h> 
#include <stdio.h> 
#include <string.h> 

int main(int argc, char **argv) { 
    char name [6]={'a','m', 'i', 'n','f','e'}; 
    char *temp= (char *) malloc(sizeof(char)*5); 
    memcpy(temp,name1,4); 
    memcpy(temp+4,name1+5,1); 
    int i; 
    for (i=0; i<5; i++) { 
     printf("%c\n",*temp); 
     temp++; 
    } 
    free(temp); 
} 

Вывод программы: амин

Answer 1

Использование

free(temp-i) 

Вы увеличиваем температуру в контуре. Следовательно, вы не освобождаете указатель, который вы создали.

Лучше было бы

for(i=0;i<5;i++){ 
    printf("%c\n",*(temp+i)); 
} 
free(temp); 

Answer 2

char name [6]={'a','m', 'i', 'n','f','e'}; 

Обратите внимание, что это не строка для C, потому что не терминатор строки символ «\ 0» (так что это просто массив символы, но не строка).

Кроме того

for (i=0; i<5; i++) { 
     printf("%c\n",*temp); 
     temp++; 
    } 

вы увеличиваете указатель (доступ к Экстерн части памяти) Вы не написать эту

for (i = 0; i < 5; i++) { 
    printf("%c", temp[i]); 
    } 
printf("\n"); 

Или лучше

printf("%s\n", temp);