double free or corruption

Хотя говно.

Покажи весь код и stacktrace

Чем это лучше?

Слишком мало кода чтобы что-то можно было утверждать. Где прототип объекта хотя бы?

Проверка на нул не нужна. И вообще

class ioid ... {
...
    std::unique_ptr<...> msgbuf;
...
};

Все, разобрался, ошибка была совсем детской.

Рассказывай, что за ошибка была? Вдруг другим поможет!

~ioid()
delete msgbuf
выдает ошибку на delete.

ссзб. надо так:

template <typename T>
class ioid
{
  std::unique_ptr <T []> msgbuf_ref;
  T *msgbuf;
  size_t msglen;
  

  public:
    ioid() : msgbuf_ref(), msgbuf(msgbuf_ref.get()), msglen(0) {}
    void resize_msg_naprimer(size_t t) {
       msgbuf_ref.reset(new T [t]);
       msgbuf = msgbuf_ref.get();
       msglen = t;
    }
};

делиты не нужны. деструкторы здесь тоже.

надо так:

std::vector<T> msgbuf;

fixed

в других местах я память не освобождаю

зато наверное пишешь куда не положено

Я сперва выделил память, затем скопировал всю переменную типа ioid, затем вышел из функции (и тут память очистилась в первый раз), а затем стал освобождать и копию.

В данном случае так не годится.

почему?

мне его потом по сетке передавать надо.

Да какая разница. Поинт был в том, что делиты не нужны.

send(skt, &(msgbug[1]), msglen, NULL) сработает. Инфа 1460‰.

мне его потом по сетке передавать надо.

http://www.cplusplus.com/reference/vector/vector/data/

или бери указатель на первый элемент, только проверяй, что вектор не пустой

Да какая разница.

кода меньше

msglen = -1;
msgbuf = NULL;

В чем профит обнулять данные в деструкторе?

&(msgbug[1]

&msgbuf.front()

А нужно

MPI_Send(msgbuf, msglen, MPI_DOUBLE, server_id, GD_POSKAS, MPI_COMM_WORLD);

Особенность MPI в том, что требуется именно массив в смысле C или фортрана. Остальные массивы могут и не заработать.

Особенностью std::vector является то, что он именно массив в смысле С или фортрана.

Остальные массивы могут и не заработать.

nOOb.

Я сперва выделил память, затем скопировал всю переменную типа ioid, затем вышел из функции (и тут память очистилась в первый раз), а затем стал освобождать и копию.

А ты уверен, что тут должен быть delete, а не delete[]? Ну и про умные указатели и вектор уже сказали.

Особенностью std::vector является то, что он именно массив в смысле С или фортрана.

Это в спецификации так написано, или «ну так оно внутри устроено»?

Это в спецификации так написано, или «ну так оно внутри устроено»?

http://en.cppreference.com/w/cpp/container/vector/data

Just like arrays, vectors use contiguous storage locations for their elements, which means that their elements can also be accessed using offsets on regular pointers to its elements, and just as efficiently as in arrays. But unlike arrays, their size can change dynamically, with their storage being handled automatically by the container.

Гуголь занёс тебя в чёрный список? За что?

ну тогда ок

Не понятно, зачем тебе тут проверка длины.

if(msgbuf)
{
    delete[] msgbuf;
    msgbuf = NULL;
}

а лучше макросы завести

#define safe_delete(ptr) do if(ptr) { delete ptr; ptr = NULL; } while(0)
#define safe_delete_array(ptr) do if(ptr) { delete[] ptr; ptr = NULL; } while(0)

а у тебя непонятно зачем проверка if(ptr)

потому что если ptr == 0, то удалять нечего, что тут непонятного?

потому что если ptr == 0, то удалять нечего, что тут непонятного?

Внутри делейта эта проверка и так есть.

Не хочешь, не проверяй. Никто не заставляет. Я лично не вижу смысла в выполнении двух операторов когда в этом нет необходимости.

хочешь, не проверяй. Никто не заставляет. Я лично не вижу смысла в выполнении двух операторов когда в этом нет необходимости.

С таким идиотским подходом, ты можешь написать так:

if (msgbuf != NULL)
  if (msgbuf != NULL)
    if (msgbuf != NULL)
      if (msgbuf != NULL)
        delete msgbuf;

тогда когда в этом нет необходимости не будет выполняться ещё больше операторов!

Габровец приехал в Софию к сыну, который учился в училище, а тот хвалится:
– Папа, сегодня я сэкономил два лева!
– Великолепно, сынок, но как тебе удалось сэкономить?
– Сегодня утром я не сел в трамвай, а чтобы не опоздать на занятия, бежал за трамваем до училища.
– Балбес! – отругал его отец. – Ты что не мог бежать за каким-нибудь такси, чтобы сэкономить больше?

[наркомане_моде]

кстати, делит сам на ноль проверяет ещё, так что одну проверку можно убрать:

if (msgbuf != NULL)
  if (msgbuf != NULL)
    if (msgbuf != NULL)
      //if (msgbuf != NULL)
        delete msgbuf;

(тред не читал) Продолбал закрыть или определить оператор копирования и копирующий конструктор, да?

Юзай std::shared_ptr, если копирование нужно, и std::unique_ptr, если нет.

И да, по твоему коду: delete для NULL разрешен и не делает ничего. Так что

 if (msglen != -1 && msgbuf != NULL)

 if (msglen != -1)

Да и вообще проверки в деструкторе сомнительны в своей нужности.

if (msgbuf != NULL)
  if (msgbuf != 0)
    if (msgbuf != false)
      if (msgbuf != nullptr)
        if (msgbuf != (void *)0)
           delete msgbuf;

А то мало ли что...

А ты уверен, что тут должен быть delete, а не delete[]?

Стандартный delete[] вызывает delete.

Стандартный delete[] вызывает delete.

И что? По твоему, «delete[]» не нужен?

В контексте этого топика - это вряд-ли была причина проблемы.

С таким идиотским подходом, ты можешь написать так:

но написал ты. Можешь с уверенностью считать себя идиотом.

К.О. напоминает, что дело не в выделении/освобождении памяти, а в вызове деструкторов.

К.О. также напоминает, что кроме std::vector существует более брутальный std::valarray - как раз для всяких ваших MPI, Си и прочих фортранов)

Твой КО опа читал?

Я сперва выделил память, затем скопировал всю переменную типа ioid, затем вышел из функции (и тут память очистилась в первый раз), а затем стал освобождать и копию.

Из кода неявно видно, что это простой байтовый буфер, а не коллекция объектов.

Твой КО опа читал?

зачем всего то? он читал тебя, который писал:

(тред не читал)

тащем-то кэп обобщает.

Проверка на msgbuf != NULL лишняя.
Где до до деструктора вы делаете delete msgbuf или где то до delete портится память.
В любом случае valgrind вам не помешает.

В чем профит обнулять данные в деструкторе?

Ну может msglen и msgbuf статиком сделаны.

В контексте этого топика - это вряд-ли была причина проблемы.

Ну да. Но я всё равно не понял смысл замечания, которое (на мой взгляд) выглядит как «можно всегда использовать простой delete».

Я говорил в контесте «в данном случае дело не в этом».