c - 将元素添加到链表？

Question

这是我的结构和方法，但它不起作用。谁能帮我解决问题在哪里？谢谢

这是结构：

    struct album
{
    char singerName[30];
    char year[4];
    char title[30];

    char songName[50];
    char songLength[50];
    struct album *next;
};
struct album *a=NULL;

这是方法：

struct album *addAlbum(struct album *list,char* year,char *title,char *singerName)
{
    struct album *temp;
    temp =(struct album*) malloc(sizeof(struct album));
    strcpy(temp->singerName,singerName);
    strcpy(temp->title,title);
    strcpy(temp->year,year);
    temp -> next = NULL;

    if(list==NULL)
    {
        return temp;
    }
    else
    {
   temp->next=list;
        return temp;
    }
}

Answer 1

对目标缓冲区的控制不足。

year如果输入为“2013” ​​，则以下操作可能会失败。这是一个字符串，需要 4+1 个字节。

char year[4];     
...
strcpy(temp->year,year);

简单的解决方法是使用char year[5]. 但这会把罐子踢到路上。

更好用strncpy(temp->year,year, sizeof(temp->year)-1); temp->year[sizeof(temp->year)-1] = '\0'。存在其他选项来防止溢出。