Как правильно очистить память?

Question

[BuckvicK]

void separator_clean(char** text, char* separator){
	int i = 0, j = 0, check = 0, k = 0;
	char *str, *buffer;
	while (i<STR){
		str = (char*)malloc(sizeof(char));
		*str = '\0';
		buffer = (char*)malloc(sizeof(char));
		*buffer = '\0';
		strcpy(str, *(text+i));
		j = 0;
		check = 0;
		while (*(str+j) != '\0'){
			buffer = (char*)malloc(sizeof(char));
			*buffer = '\0';
			if (strchr(separator, *(str+j)) != NULL){
				check++;
			}
			else {check = 0;}

			if ((check > 1)){
				*(str + j - 1) = *separator;
				strncat(buffer, str, j);
				printf("#BUF:%s\n", buffer);
				strcat(buffer, str + j+1);
				printf("#BU2:%s\n", buffer);
				strcpy(str, buffer);
				printf("#STR:%s\n", str);
				k = 0;
				j--;
			}
			j++;
			//free(buffer);
		}
		strcpy(*(text + i), str);
		printf("STR:%s\n", str);
		printf("TEX:%s\n", *(text+i));
		printf("BUF:%s\n", buffer);
		i++;
		//free(str);
	}
}

Функция получает массив текста(text), и массив разделителей(separator). Выполняет операции построчно, через str.
Она заменяет несколько подряд идущих разделителей на первый в массиве разделителей. Работает правильно, но возникает проблема с памятью, если строка достаточно длинная. При освобождении памяти(закоментированные строчки), программа выдает ошибку: "Has triggered a breakpoint.".
Объясните в чем проблема?

Answer 1

[MiiNiPaa]

buffer = (char*)malloc(sizeof(char));Выделяет память для хранения одного символа.
Пишется туда больше одного символа, получаем классическое переполнение буфера, которое очевидно повреждает какую-то информацию необходимую для корректного удаления.
+ буфер созданый на строке 7 никогда не освобождается

И чем вам strspn/strcspn не угодили?

Comments

[BuckvicK]

А разве принудительно функции strncat не расширят buffer?

Извините, но я не понимаю как тут можно использовать strspn/strcspn, только учусь)

[MiiNiPaa]

> А разве принудительно функции strncat не расширят buffer?
Нет. Читайте описание функций перед их использованием.
en.cppreference.com/w/c/string/byte/strncat

The behavior is undefined if the destination array does not have enough space for the contents of both dest and the first count characters of src, plus the terminating null character.

В C почти все функции стандартной библиотеки не выделяют память динамически. То есть оперируют на выданном буфере или сохраняют данные в какую-то статическую память которая перезапишется при следующем вызове функции.

> я не понимаю как тут можно использовать strspn/strcspn
Находите количество символов (n) в слове (не в массиве разделителей) используя strcspn. Копируете эти n символов, прибавляете n к указателю на оригинальную строку.
Находите количество символов-разделителей (m) при помощи strspn, копируете первый из них, прибавляете m к указателю на оригинальную строку.
Повторить.
Так пока не наткнётесь на нулевой символ.