Как грамотно выделить память?

Question

[pentagon9714]

Здравствуйте!
Столкнулся со следующей проблемой - данная функция отлично работает в режиме отладки, но при обычном запуске в сборке release/debug вылетает с неизвестной ошибкой.

void inputToList()
{
	int i = 0;                                
	link current = list_watch;
	while (input[i] != '\0')
	{
		current->data = (char*)malloc(buf*sizeof(char));
		current->datalen = 0;
		while (input[i] != '\n')
		{
			current->data[current->datalen] = input[i];
			current->datalen++;
			if (current->datalen%buf == 0)
			{
				current->data = (char*)realloc(current->data, (buf + current->datalen)*sizeof(char));
			}
			i++;
		}
		current->data[current->datalen] = '\0';
		i++;
		blockscount++;
		if (input[i] != '\0')
		{
			current->next = (link)malloc(sizeof(link));
			current = current->next;
		}
		else
		{
			current->next = NULL;
		}
	}
}

Comments

[Арсений Ефремов]

Прикрепите текст ошибки что ли.

[Денис Загаевский]

Во-первых, зачем столько реаллоков? Это вредно. Во-вторых, если строка будет кончаться \n, вы вылезете за её пределы.

Answer 1

[Армянское Радио]

У вас как минимум переменная не инициализирована - list_watch.
Если это глобальная переменная, все становится намного хуже.

Проверка на ошибки также отсутствует - добавляйте.

Comments

[pentagon9714]

list_watch глобальная. Но почему от этого всё становится намного хуже?

[Армянское Радио]

pentagon9714: Глобальные переменные нарушают структуру программы, требуют организации контракта, который компилятор проконтролировать не в состоянии, а в случае C++ создают огромные проблемы с порядком конструкциии и деструкции.

[Владимир Дубровин]

Армянское Радио: глобальные переменные в C всегда статические, а потому всегда инициализированы (по стандарту C89).

[Армянское Радио]

Владимир Дубровин: Я написал - в случае C++. В случае C эта проблема остро не стоит, зато есть другая - программа превращается в лапшу.

[Владимир Дубровин]

Армянское Радио: вы удивитесь, но в случае C++ все ровно так же. На самом деле, статические переменные (включая глобальные) создаются в момент запуска приложения, поэтому мусора там вообще быть не может - все свежевыделенные страницы памяти всегда занулены, иначе одно приложение могло бы читать данные оставшиеся от другого приложения.

[Армянское Радио]

Владимир Дубровин: Не забывайте, что в C++ переменная может быть классом. А порядок инициализации глобальных экземпляров классов стандартом не регламентирован.

По поводу обнуления страниц - не забывайте, что C++ переносим между платформами. Под каким-нибудь DOS или на микроконтроллере страница может оказаться не пустой.

Answer 2

[Петр]

У вас
while (input[i] != '\n') i++;
А если этого '\n' нету, то вы пропуская '\0' вылетаете за строку.

Comments

[Денис Загаевский]

А если есть и он последний - то всё равно вылетает за строку.

[Петр]

Денис Загаевский: Если это null-терминейтед строка. То дальнейшие проверки не позволяют вылететь.