_CRTLISVALIDHEAPPOINTER在破坏过程中的错误

我，当我的显示对象t_display被解构时，会遇到_CrtlIsValidHeapPointer(block)错误。这发生在Animation.cpp中的while循环之后，用于获取用户输入并进行多个显示。

我知道这是因为我为char * p_name分配内存，然后将该指针存储在对象中，但我不确定如何解决它。我必须使用char *作为显示对象名称，所以这必须意味着我必须为其分配内存。

我认为可能有两个问题之一。1）我正在为char *分配错误或错误地复制字符串的内存2）我写了错误的distuructor

在这两种情况下，我都不确定如何解决我遇到的错误，希望您可以指向正确的方向。

animation.cpp

#include <crtdbg.h>
#include <iostream>
#include <string>
#include <vector>
#include <forward_list>
using namespace std;
#include "Display.h"
#include "Frame.h"
#include "Animation.h"

void Animation::InsertFrame() {
    int numDisplays; //for user input of display number
    vector <Display>v; //vector for containing display objects
    int p_x; //will contain user input for pixel_x
    int p_y; //will contain user input for pixel_y
    int p_duration; //will contain user input for duration
    char * p_name; //temp string to contain user input for name
   //will contain p_name to be passed to display constructor
    string frameName; //contains user input for the frame name
    int q = 0; //used to count the diplay #
    //begin reading user input
    cout << "Insert a Frame in the AnimationnPlease enter the Frame filename: " ;
    cin >> frameName;
    cout << "Entering the Frame Displays (the sets of dimensions and durations) " << endl;
    cout << "Please enter the number of Displays: " ;
    cin >> numDisplays;
    string d_name;
    //display creation loop for # of displays entered
    while (numDisplays > 0) {
        //char * name=nullptr;
        cout << "Please enter pixel x for Display #"<<q<<" pixel_x:";
        cin >> p_x;
        cout << "Please enter pixel y for Display #"<<q<<" pixel_y:" ;
        cin >> p_y;
        cout << "Please enter the duration sec for this Display: " ;
        cin >> p_duration;
        cout << "Please enter the name for this Display: " ;
        //cin >> p_name;
        cin >> d_name;
        //p_name = new char[strlen(name)];
        p_name = new char[d_name.length() + 1]; //allocate for the size of the name entered
        strcpy(p_name, d_name.c_str()); //copy string to char []
        Display t_display =  Display(p_x, p_y, p_duration, p_name); //make a new display with the user input values
        v.push_back(t_display); //pushing onto the vector
        numDisplays--;
        q++;
    }

display.h

// Display.h
#pragma once
class Display
{
    int pixel_x;
    int pixel_y;
    int duration;
    char* name;
public:
    Display(int x, int y, int duration, char* name);
    Display(const Display&);
    ~Display();
    friend ostream& operator<<(ostream&, Display&);
};

display.cpp

#include <crtdbg.h>
#include <iostream>
#include <string>
#include <vector>
#include <forward_list>
using namespace std;
#include "Display.h"
Display::Display(int x, int y, int d, char* n):pixel_x(x), pixel_y(y), duration(d), name(n) {
}
Display::Display(const Display& p) {
    //copy values from p
    pixel_x = p.pixel_x;
    pixel_y = p.pixel_y;
    duration = p.duration;
    name = p.name;
}
Display::~Display() {
}

该程序无需驱动器而起作用，但当然具有内存泄漏，这是不可接受的。当我添加一个简单的破坏者时，例如：

if(name){
 delete[] name;
}

它将丢弃该错误。