free

@VeNick · Регистрация: 02.11.2013

Author24 — интернет-сервис помощи студентам

Выделил память под массив(строку), размером 5 элементов.

C++

    char *str;
    str = (char*)calloc(5,sizeof(char));
    if (str == NULL)
    {
        cout<<"ERROR"<<endl;
        exit(1);
    }

По мере ввода строки, выделяется дополнительная память.
realloc может возвратить указатель на новую область памяти, поэтому старую нужно очистить.

C++

    int i;
    char simvol;
    char *strold;
    for (i = 0; (simvol = getchar()) != '\n'; i++)
    {
        if (i%5 == 0  && i != 0)
            {
                strold = str;
                str = (char*)realloc(str, (i+5)*sizeof(char));
                if (str == NULL)
                {
                    cout<<"ERROR"<<endl;
                    exit(1);        
                }
                if(strold != str)
                    free(strold);
            }
        str[i] = simvol;
    }

С помощью отладочных печатей, обнаружил, что ошибка возникает в строчке

C++

1	free(strold);

Что-то там про кучу...
Помогите разобраться.

@gray_fox · 04.11.2013, 12:05

realloc может возвратить указатель на новую область памяти, поэтому старую нужно очистить.

Не нужно очищать в этом случае; нужно освобождать память только после того, как она уже не нужна.

@VeNick · 04.11.2013, 12:15 **[ТС]**

Сообщение от gray_fox

Не нужно очищать в этом случае; нужно освобождать память только после того, как она уже не нужна.

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

@grizlik78 · 04.11.2013, 12:23

Сообщение от VeNick

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

Потому, что realloc эту память уже освободил.

Tulosba · 04.11.2013, 12:29

Сообщение от VeNick

realloc может возвратить указатель на новую область памяти, поэтому старую нужно очистить.

Вторая часть предложения ошибочна.

If the new size of the memory object would require movement of the object, the space for the previous instantiation of the object is freed.

@VeNick · 04.11.2013, 12:30 **[ТС]**

Сообщение от grizlik78

Потому, что realloc эту память уже освободил.

Понятно, спасибо.

@gray_fox What a waste! 1608 / 1300 / 180 Регистрация: 21.04.2012 Сообщений: 2,729
	04.11.2013, 12:05	2
	realloc может возвратить указатель на новую область памяти, поэтому старую нужно очистить. Не нужно очищать в этом случае; нужно освобождать память только после того, как она уже не нужна. 1

@VeNick 0 / 0 / 0 Регистрация: 02.11.2013 Сообщений: 5
	04.11.2013, 12:15 [ТС]	3
	Сообщение от gray_fox Не нужно очищать в этом случае; нужно освобождать память только после того, как она уже не нужна. Но если данные в этой области памяти не имеют смысла, потому что в указателе лежит уже другой адрес, почему бы не освободить там память? Видимо, я просто чего-то элементарного не понимаю. 0

@grizlik78 2381 / 1665 / 279 Регистрация: 29.05.2011 Сообщений: 3,399
	04.11.2013, 12:23	4
	Сообщение от VeNick Но если данные в этой области памяти не имеют смысла, потому что в указателе лежит уже другой адрес, почему бы не освободить там память? Потому, что realloc эту память уже освободил. 1

Tulosba :) 4773 / 3267 / 497 Регистрация: 19.02.2013 Сообщений: 9,046
	04.11.2013, 12:29	5
	Сообщение от VeNick realloc может возвратить указатель на новую область памяти, поэтому старую нужно очистить. Вторая часть предложения ошибочна. If the new size of the memory object would require movement of the object, the space for the previous instantiation of the object is freed. 1

@VeNick 0 / 0 / 0 Регистрация: 02.11.2013 Сообщений: 5
	04.11.2013, 12:30 [ТС]	6
	Сообщение от grizlik78 Потому, что realloc эту память уже освободил. Понятно, спасибо. 0