неправильно копирование в структуры

Question

Я передаю строку в структуры, я делаю это так внутри цикл

printf("copy = %s\n",copy_p); 
str[i].string=(char*)malloc(strlen(copy_p)+1 * sizeof(char)); 
strcpy(str[i].string,copy_p); 
printf("skop = %s\n" , str[i].string); 

Так что если copy_p переменная «Программа перестала работать» это то, что происходит

printf("copy = %s\n",copy_p); // copy = Program has stopped working 
printf("skop = %s\n" , str[i].string); // skop = Program has stopped working 

Но если я назову это printf("%s\n",str[0].string), он выдает Program has stopped work!, почему это так? но оно не нравится все это время он работает для большинства входов

Answer 1

Эта строка неверна независимо от типа copy_p:

str[i].string=(char*)malloc(sizeof(copy_p)+1 * sizeof(char)); 

• Если copy_p является массив символов инициализируется строковым, т.е.

  char copy_p[] = "Program has stopped working"; 
  

  затем +1 ненужно, так как размер массива уже включает в себя нулевой терминатор.

• Если copy_p указатель char *copy_p, то вам нужно позвонить strlen вместо sizeof, т.е.

  str[i].string=malloc(strlen(copy_p)+1 * sizeof(char)); 
  

Примечание: результаты литья из malloc ненужно в С.