String is not null terminated

String String::operator+ (const String &right)
{
    String temp;
    temp.length = this->length + right.length + 1;
    delete [] temp.sPtr;
    temp.sPtr = new char[temp.length];
    assert(sPtr != 0);
    strcpy_s(temp.sPtr, this->length + 1, this->sPtr);
    strcat_s(temp.sPtr, right.length + 1, right.sPtr);
    return temp;
}

String String::operator+ (const String &right)
{
    String temp;
    temp.length = this->length + right.length;
    delete [] temp.sPtr;
    temp.sPtr = new char[temp.length + 1];
    assert(sPtr != 0);
    strcpy_s(temp.sPtr, temp.length + 1, this->sPtr);
    strcat_s(temp.sPtr, temp.length + 1, right.sPtr);
    return temp;
}

const String &String::operator= (const String &right)
{
    if (&right != this)
    {
        delete [] sPtr;
        length = right.length;
        sPtr = new char[length + 1];
        assert(sPtr != 0);
                //вот эта следующая строка выдает Buffer is too small
        strcpy_s(sPtr, length + 1, right.sPtr);
    }
    return *this;
}

char *sPtr = new char[x];

const String &String::operator= (const String &right)
{
    if (&right != this)
    {
        delete [] sPtr;
        length = right.length;
        sPtr = new char[length + 1];
        assert(sPtr != 0);
                //следующай строка не работает
        strcpy_s(sPtr, length + 1, right.sPtr);
    }
    return *this;
}

String String::operator+ (const String &right)
{
    String temp;
    temp.length = this->length + right.length;
    delete [] temp.sPtr;
    temp.sPtr = new char[temp.length + 1];
    assert(sPtr != 0);
    strcpy_s(temp.sPtr, temp.length + 1, this->sPtr);
    strcat_s(temp.sPtr, temp.length + 1, right.sPtr);
    return temp;
}

String temp;
return temp;

String String::operator+ (const String &right)
{
    String *temp = new String();
    temp->length = this->length + right.length;
    delete [] temp->sPtr;
    temp->sPtr = new char[temp->length + 1];
    assert(sPtr != 0);
    strcpy_s(temp->sPtr, temp->length + 1, this->sPtr);
    strcat_s(temp->sPtr, temp->length + 1, right.sPtr);
    return *temp;
}

#include <iostream>
using namespace std;
 
class Vec2 {
    int x,y;
public:
    Vec2() {x=y=0;}
    Vec2(int i, int j) {x=i; y=j;}
    //Vec2(Vec2 &v) {x=v.x; y=v.y;}
    Vec2 operator+(Vec2 op2);
    Vec2 operator=(Vec2 op2);
    void show();
};
 
Vec2 Vec2::operator+(Vec2 op2) {
    Vec2 temp;
    temp.x=x+op2.x;
    temp.y=y+op2.y;
    return temp;
}
Vec2 Vec2::operator=(Vec2 op2) {
    x=op2.x;
    y=op2.y;
    return *this;
}
void Vec2::show() {
    cout << x << ' ' << y << '\n';
}
 
int main() {
    Vec2 a(1,2), b(3,-1),c(-5,2), d;
    d=a+b+c;
    a.show();
    b.show();
    c.show();
    d.show();
    return 0;
}