如何调试＆＃34;调试断言失败。缓冲太小＆＃34;？

Question

当我使用strcpy_s时，始终显示相同的错误Debug Assertion failed. L Buffer is too small &&0。有人可以帮我解决吗？我使用的是Microsoft Visual Studio Ultimate 2012

struct Nod{
    char *Number;
    char *Name;
    Nod *drt, *stg;
};



void Insert(Nod *&p, char* n, char nr [])
{
    if(p==0 )
        p=makeNod(n, nr);
    else
    {
        ...
    }   
}

Nod * makeNod(char *name, char nr[])
{
    Nod *p=new Nod;
    p->Name=new char [strlen(name)+1];
    strcpy_s(p->Name, strlen(name), name);  //Assertion failed          
    strcpy_s(p->Number, strlen(nr), nr);
    p->stg=p->drt=0;
    return p;  
}
int main()
{

    Nod *p=0;
    int c;
    char nr[9];
    char*name=new char [20];
    cin >> c;
    while(c!=0)
    {
        cout << "Name:  "<< endl;
        cin >> name;
        cout << "Number:  "<< endl;
        cin >> nr;
        Insert(p, name, nr);
        cin >> c;
    }
    return 0;
}

Answer 1

首先，调试模式下的strcpy_s检查指定的长度（strlen(name）是否不足以容纳name +终止\0，因此它断言。

其次，您没有为p->Number预留缓冲区。

    Nod *p=new Nod;
    p->Name=new char [strlen(name)+1];
    strcpy_s(p->Name, strlen(name)+1, name); // <-- +1, to not miss the null character at the end
    p->Number=new char [strlen(nr)+1];      // <-- You missed this
    strcpy_s(p->Number, strlen(nr)+1, nr);

最后，使用std::string并保存这些令人头疼的问题。

Answer 2

strcpy_s()的第二个参数似乎是指向第一个参数的缓冲区的大小。由于字符串副本需要复制strlen(name)个字符和一个终止空字符，因此您需要提供一个至少一个大于{的字符的缓冲区{1}}。实际上，你做了分配一个适当大小的缓冲区！您刚刚没有告知strlen(name)这个事实。

此外，一旦你克服了这个错误，你会发现你没有为strcpy_s()分配任何内存。我还建议实际保护Number的结果，因为操作可能很昂贵，例如，当字符串非常长时。