C++新手上路，求解这段代码的两个错误及原因

如题，一共两个错误，求解答。
感谢！

个人认为两个错误应该是如下两个：
1、函数一进来应该立马判断 source 和 this 是否相等（因为重载赋值运算符的时候必须要考虑自己给自己赋值的场景），如果二者相等则直接返回*this. 不必再做申请内存和赋值的动作。
2、在 执行 m_data=newArray之前应该要先判断m_data是否为NULL, 如果m_data不是NULL要先讲m_data所指向的内存释放掉，避免内存泄露。
改正后的代码应该是这样的：
template
Array& Array::operator = (const Array& source)
{
if(this != &source)
{
T* newArray = new T[source.size];
if(m_data != NULL)
{
delete[] m_data;
}
Size = source.size;
m_data = newArray;

    for(int i=0; i< std::min(Size, source.size), ++i)
        m_data[i] = source.m_data[i];

}
return *this;

}

楼主要是对这个回答感到满意的话，记得采纳哦，急着等C币用，谢谢啦~~~

m_data没有定义
m_data没有释放，造成内存泄漏

 template <class T>
Array<T>& Array<T>::operator = (const Array<T>& source)
{
    if(this != &source)
    {
        T* newArray = new T[source.size];
        if(m_data != NULL)
        {
            delete[] m_data;
        }
        Size = source.size;
        m_data = newArray;

        for(int i=0; i< std::min(Size, source.size), ++i)
            m_data[i] = source.m_data[i];

    }
    return *this;
}