2
#include <iostream>
#include <cstring>
using namespace std;

char slot_desc[] = {'a', 'b', 'c', 'd', 'f', 'e', 0, 0, 0, 0, 0};

typedef struct SlotInfo
{
    char slotDescription[10];   

}SLOT;

struct GeneralSlot
{
    SLOT *SlotInf;
};

GeneralSlot *Slot1;

int main(int argc, char* argv[])
{

    Slot1->SlotInf = NULL;
    memmove(Slot1->SlotInf->slotDescription, slot_desc, 5);

    return 0;
}

В данном коде получаю Ошибка сегментирования (сделан дамп памяти). Компилирую под Linux. Что я делаю не так или чего-то не понимаю с указателями? При запуске от имени суперпользователя ошибку не выдает, но все равно копирование не выполняется. Подскажите, пожалуйста, в чем ошибка.

4
  • 4
    Вы задаете Slot1->SlotInf=NULL и тут же пытаетесь обратится к полую по этому указателью, т.е. по факту делаете NULL->slotDescription
    – Mike
    5 мая 2017 в 10:20
  • @Mike, ответы в ответы?
    – Qwertiy
    5 мая 2017 в 10:21
  • Хотя ошибка еще раньше происходит. Вы определяете указатель Slot1 но он изначально никуда не смотрит. Надо выделить память для хранения структуры GeneralSlot и присвоить адрес этой области памяти этому указателю
    – Mike
    5 мая 2017 в 10:21
  • @Mike, спасибо, разобрался!
    – VadoM
    5 мая 2017 в 12:07

2 ответа 2

4

Вы совершаете одну и ту же ошибку дважды.

В этом предложении объявлен указатель в глобальном пространстве имен

GeneralSlot *Slot1;

Компилятор инициализирует этот указатель нулем. Фактически это объявление эквивалентно объявлению

GeneralSlot *Slot1 = NULL;

То есть данный указатель не указывает ни на какой реальный объект типа GeneralSlot. Поэтому попытка разыменовать такой указатель в предложении

Slot1->SlotInf = NULL;

ведет к неопределенному поведению программы.

Даже если бы указатель Slot1 указывал бы на реальный объект, тем не менее вы повторяете предыдущую ошибку, инициализировав уже другой указатель Slot1->SlotInf этого объекта нулем, и пытаясь что-то записать в память по нулевому адресу

memmove(Slot1->SlotInf->slotDescription, slot_desc, 5);

Вам надо, чтобы оба указателя указывали на реальные объекты. Вы могли бы написать, к примеру,

// ...

GeneralSlot *Slot1;

int main(int argc, char* argv[])
{

    Slot1 = new GeneralSlot;

    Slot1->SlotInf = new SLOT;

    memmove(Slot1->SlotInf->slotDescription, slot_desc, 5);

    // ...

    delete Slot1->SlotInf;
    delete Slot1;

    return 0;
}
3
  • спасибо за подробное объяснение. С указателями мало знаком, поэтому и туплю на ровном месте)
    – VadoM
    5 мая 2017 в 12:09
  • Если я правильно помню, компилятор не обязан инициализировать глобальные переменные нулём, если они не объявлены как статические.
    – 0andriy
    6 мая 2017 в 10:52
  • @0andriy Все глобальные переменные имеют статическую память,, которая инициализируется нулем. Переменные, объявленные с ключевым словом static, имеют внутреннее связывание. Тем не менее, они имеют статическую память. 6 мая 2017 в 13:00
3

Итак, глобальная переменная

GeneralSlot *Slot1;

т.е. инициализированная нулем. После чего

int main(int argc, char* argv[])
{
    Slot1->SlotInf = NULL;

И куда вы записываете этот NULL? В NULL?
Вот и получаете то, что получаете...

Выделите память для GeneralSlot, на которую будет указывать Slot1.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.