0

При вызове Func() заполняется buffer данными из сокета, после происходит их конвертация в класс Derived. Если после конвертаци вызвать функию Foo(), то происходит ошибка сигментации. Если создавать класс в стеке, то ошибка не происходит.

header file

class Interface
{
public:
    virtual char Foo() = 0;
};

class Derived : public Interface
{
public:
    char Foo() override
    {
        return val[0];
    }
    char val[10];
};

source file

boost::array<char, 8192> buffer;

void Func()
{
...
auto derived = std::make_shared<Derived>();

 std::memcpy(derived.get(), buffer.data(), sizeof(Derived));
 derived->Foo(); // Segmentation fault
...
}
9
  • что-то не видно метода get в вашем Derived 9 авг 2018 в 12:04
  • @AndrejLevkovitch, derived это shared_ptr а не Derived
    – acade
    9 авг 2018 в 12:05
  • а, точно, затупил 9 авг 2018 в 12:07
  • 1
    std::memcpy(derived->val, buffer.data(), sizeof derived->val)
    – acade
    9 авг 2018 в 12:21
  • 1
    std::copy_n(buffer, sizeof(derived->val), derived->val); 9 авг 2018 в 12:53

1 ответ 1

3

Вообще, ИМХО, так лучше никогда не делать: записывать в объект побайтово какую-то последовательность байт, беря при этом указатель на начало объекта. Вот смотрите: если бы у вас это был простенький класс, то все бы работало как вы и ожидали (на сколько я могу судить), но вот вы унаследовались... Тоже вроде все должно работать, но у вот только у вас есть виртуальный метод - это все меняет. Можете сами убедится, для этого просто проверьте sizeof. Думаю вы удивитесь, тому что размер оказался больше, чем если бы не было виртуального методы. Дело в том, что компилятор должен как-то определять, какой метод дернуть, а если у вас есть виртуальные методы то как это сделать? Раз так, объект класса должен хранить указатель на таблицу функций (извиняюсь за неточность формулировки), которые он может и должен вызывать. Получается что вы переписываете указатель на эту таблицу, и объект обращается по невалидному указателю.

Вообщем: либо используйте метод для записи (собственно для чего и создавались классы), либо берите указатель именно на член класса, который нужно изменить и перезаписывайте исходя из его размера. Но второй вариант абсолютно претит принципам инкапсуляции.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

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