Ошибка при перегрузке оператора +

String operator+(String & a, String & b)
{
    String c;
    c.i = a.i + b.i;
    c.p = new char[c.i + 1];
    strcat(c.p, a.p);
    strcat(c.p + a.i, b.p);
    return c;
}

strcpy(c.p, a.p); 
strcat(c.p, b.p);

#include <string>
#include <iostream>
 
using namespace std;
 
class String
{
public:
    String();
    String(char[]);
    String(String &);
    
    ~String();
 
    friend istream & operator >> (istream & is, String  &);
    friend ostream & operator << (ostream & is, String  &);
    friend bool operator > (String &, String &);
    friend bool operator < (String &, String &);
    friend String operator+ (String &, String &);
    String & operator= (String &);
 
 
    int Lange();
 
private:
    int i;
    char *p;
};

#include <iostream>
#include <string>
#include <cstring>
#include <cctype>
#include <fstream>
#include "Header.h"
#include <sstream>
 
using namespace std;
 
String::String()
{
    i = 1;
    p = '\0';
}
 
String::String(char s[])
{
    i = strlen(s);
    strcpy(p, s);
}
String::~String()
{
    delete p;
}
String::String(String & Str)
{
    p = new char[Str.i + 1];
    strcpy(p, Str.p);
}
 
 
istream & operator>>(istream & is, String & Str)
{
    char temp[1000];
    is.getline(temp, 1000);
    Str.i = strlen(temp);
    Str.p = new char[Str.i];
    strcpy(Str.p, temp);
    return is;
}
ostream & operator<<(ostream & is, String & Str)
{
    is << Str.p;
    return is;
}
bool operator>(String & a, String & b)
{
    if ((*a.p) > *(b.p))
        return true;
    else
        return false;
}
bool operator<(String & a, String & b)
{
    if ((*a.p) < *(b.p))
        return true;
    else
        return false;
}
String operator+(String & a, String & b)
{
    String c;
    c.i = a.i + b.i;
    c.p = new char[c.i + 1];
    strcpy(c.p, a.p);
    strcat(c.p, b.p);
    return c.p;
}
 
String & String::operator=(String & Str)
{
    i = Str.i;
    p = new char[i+1];
    strcpy(p, Str.p);
    return *this;
}
int String::Lange()
{
    return strlen(p);
}

#include <iostream>
#include <string>
#include "Header.h"
#include <cctype>
#include <fstream>
 
using namespace std;
 
 
 
int main()
{
    setlocale(LC_ALL, "Russian");
 
    String x, y, z;
    cin >> x;
    y = x;
    cout << y;
 
    system("pause");
    return 0;
}

String::String()
{
    i = 1; // ???
    //p = '\0';
    p = nullptr;
}

// Str.p = new char[Str.i];
Str.p = new char[Str.i + 1];

String::String(char s[])
{
    i = strlen(s);
    p = new char[i + 1];
    strcpy(p, s);
}