Что не так в перегрузке оператора индексирования?

Star& operator[](const int index);

Star& Galaxy::operator[](const int index)
{
    return stars[index];
}

Sputnik sp(name, 12, 3);
    Star star(12,3,1,name,4,sp);
    Galaxy gal(3);
    gal[0] = star;

Galaxy::Galaxy(int _data, Star* _stars, int length)
    :date(_data), length(_length), stars(new Star[length])
{
    for (int i = 0; i < length; ++i)
        stars[i] = _stars[i];
}

Galaxy::Galaxy(int _data, Star* _stars, int length)
    :date(_data), length(_length), stars(new Star[length])
{
    for (int i = 0; i < length; ++i)
        stars[i] = _stars[i];
}

Galaxy gal(12, &star, 3);
    gal[0] = star;
    gal[1] = star2;
    gal[2] = star3;

Sputnik Sputnik::operator =(const Sputnik& v)
{
    
    delete[] name;
    if (this == &v) return *this;
    name = new char[strlen(v.name) + 1];//эта ошибка здесь
    strcpy_s(name, strlen(v.name) + 1, v.name);
 
    mass = v.mass;
    radius = v.radius;
    return *this;
}

Star star(12,3,1,name,4,sp);
    Star star2(34, 5, 6, name, 12, sp);
    Star star3(66, 89, 99, name, 5555, sp);
    Galaxy gal(12, &star, 3);
    gal[0] = star;
    gal[1] = star2;
    gal[2] = star3;

    Star arr[] =
    {
        Star(12, 3, 1, name, 4, sp),
        Star(34, 5, 6, name, 12, sp),
        Star(66, 89, 99, name, 5555, sp)
    };
    Galaxy gal(12, arr, sizeof(arr)/sizeof(Star)); // звезды скопированы, ничего присваивать не нужно

Galaxy(int _data, Star* _satrs,int _lenght);

Galaxy::Galaxy(int _data, Star* _stars, int _length)
    :date(_data), length(_length), stars(new Star[length])
{
    for (int i = 0; i < length; ++i)
        stars[i] = _stars[i];
}

Sputnik sp(name, 12, 3);
    Star arr[] =
    {
        Star(12, 3, 1, name, 4, sp),
        Star(34, 5, 6, name, 12, sp),
        Star(66, 89, 99, name, 5555, sp)
    };
    Galaxy gal(12, arr, sizeof(arr) / sizeof(Star));
    cout << "Перегруженный индексатор" << endl;
    for (int i = 0; i < 3;i++)
    {
        cout << gal[i] << endl;
        
    }