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] = ''. 存在其他选项来防止溢出。