这段程序为什么会出现bug？strncpy应怎么修改才行？谢谢。

Question

void Database_set(struct Connection *conn, int id, const char *name, const char *email){struct Address *addr = &conn->db->rows[id];if(addr->set) die("Already set, delete it first");addr->set = 1;// WARNING: bug, read the "How To Break It" and fix thischar *res = strncpy(addr->name, name, MAX_DATA);// demonstrate the strncpy bugif(!res) die("Name copy failed");res = strncpy(addr->email, email, MAX_DATA);if(!res) die("Email copy failed");}

Answer 1

strncpy的参数是char*,而这个函数Database_set，name,和email的类型的是const char*
所以会出错，因为strncpy里面的实现需要操作指针位移，但是const却限制不允许修改指针，
你可以编译这段代码，一定是编译出错，而且会提示const char* 转换char*失败的问题

Answer 2

应该是指针越界吧，email的容量比MAX_DATA小的时候，就越界了，add->email同理
有溢出漏洞
应该在复制之前比较两个字符串的容量