Почему программа ломается при освобождении памяти?

Question

[Даниил]

Вот так выглядет вырезанный код,
readLine() возращает new char[length] . Потом собираюсь освободить память и все крашиться, при этом если не удалять (убрать delete [] ) то будет утечка.
В чем может быть ошибка

void run(SocketListener* _this) {
	char *ret = NULL;
	try {
		for (;;) {
			ret = _this->readLine(); // вернет new char[len];
			if (ret == NULL) break;
			std::cout << ret;
			_this->getSocket()->send(response, lenResponse);

			delete[]ret;  // !!!!!!!!!!!!!!! ЛОМАЕТСЯ
		}
		//	} while (reads_bytes > 0 /*&& len==reads_bytes*/);

	}
	catch (std::runtime_error & err) {
		//std::cout << "ERROR  "<<err.what() << std::endl;
		if (ret != NULL) delete[]ret;
	}
	delete[] buf;
}
char * _Interface_Socket::readLine()
{
	std::string line = "";
	char buf[1];
	try {
		do {
			int reads_bytes = socket->recv(buf, 1);
			if (reads_bytes < 1) return NULL;
			line.push_back(buf[0]);

		} while (buf[0] != '\n');
	}
	catch (std::runtime_error &err) {
		status = false;
		return NULL;
	}
	char *response = new char[line.size()];
	std::strcpy(response, line.c_str());
	return response;
}

Comments

[Даниил]

не знаю что за выхлоп, visual studio просто виснет, даже при откладке не чего не показывает, а просто обрывает. Приложение многопоточное.
При том _this->readLine(); всегда В онлай компиляторах все работает, то есть чем мой код отличается
от этого

char * fun(){
    char *r=new char[10];
    char *response = new char[10];
	strcpy(response, r);
    return response;
}
int main()
{
  char *ptr;		
for(int i=0;i<9;i++){
  ptr=fun();
  delete []ptr;// все норм
}
return 0;
}

[Даниил]

Я ржу с сишников, как и раньше угорал, почти все являются зазнавшимся б*длом. По любому из 10 других языков мне спокойно бы ответили, а тут уже 10 раз оскорбляют. Вообще в шоке.

[jcmvbkbc]

char * fun(){
    char *r=new char[10];
    char *response = new char[10];
  strcpy(response, r);
    return response;
}

Даниил, в массиве r находится мусор, в котором среди первых 10 элементов может быть, а может и не быть символа '\0'. Поэтому сколько символов strcpy скопирует из r в response -- неизвестно. Если больше 10, то с высокой вероятностью программа упадёт при освобождении массива response.

[Евгений Шатунов]

Даниил ,
Ржать-то можно с кого угодно, это дело дозволительное. У тебя просто базы абсолютно никакой нет и ошибки на уровне полного незнания своего вопроса.

Тыб заглянул бы сперва в документацию на std::string, глянул бы на пояснения по методу size().
Тыб сразу понял, что этот метод возвращает только число символов в строке и что упреждение на терминальный символ тебе в своем char *response = new char[line.size() + 1]; надо брать самостоятельно.

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

Пока у тебя видно только лень читать документацию по используемым инструментам. Ты все эти свои проблемы 100% способен решить самостоятельно через ознакомление с cpp-reference.

[Павел Язовских]

Даниил, использование в C++ коде new/delete и raw-строк у "зазнавшихся" плюсовиков вызывает неконтролируемые приступы ярости. Контроль ярости — вопрос отдельный и личный, но сама ярость не беспочвенна и пропорциональна человекомесяцам, проведённым в отладчике в попытках найти, где же портится указатель. Да, работать с C-style кодом плюсовик должен уметь, но писать его обязан избегать.

Если вы ещё только учитесь, то лучшим способом сейчас будет отправиться на CodeWars и до упора решать задачки 6-7 уровней, причём поставив себе цель вообще не использовать raw-массивы, raw-указатели, а использование raw-циклов (обычные do/while/for) свести к минимуму. Как это сделать см. ниже (соответствующие альтернативы гуглить или читать на cppreference):
Вместо динамических массивов и выделений памяти через new[] — std::vector, std::deque
Вместо велосипедов со списками — std::list, std::map, std::unordered_map
Вместо статических массивов — std::array
Вместо сырых строк — std::string
Вместо сырых указателей — std::shared_ptr и std::unique_ptr, итераторы
Вместо прямой работы с битами — std::bitset
Вместо сырых циклов — range-based for и по-максимуму имеющиеся стандартные алгоритмы из <algorithm> и <numeric>, такие как std::transform, std::accumulate, std::copy_if, std::generate, std::sort, std::next_permutation и т.д. Для эффективного использования стандартных алгоритмов сначала в обязательном порядке прочитать, что такое лямбда-выражения на cppreference или на хабре.

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

[Павел Язовских]

Даниил, искусственный пример, иллюстрирующий предыдущий комментарий. Задача: создать матрицу RxC, заполнить каждую строку возрастающей последовательностью, начиная с N, инвертировать столбцы и вывести на экран:
C-style решение (как не надо):

#include <iostream>
#include <vector>
void invert_cols(size_t rows, size_t cols, int n) {
	int** m = new int*[rows];
	for(size_t row = 0; row < rows; ++row) {
		m[row] = new int[cols];
		for(size_t col = 0; col < cols; ++col) {
			m[row][col] = n + col;
		}
		for(size_t col = 0; col < cols / 2; ++col) {
			int tmp = m[row][col];
			m[row][col] = m[row][cols - col - 1];
			m[row][cols - col - 1] = tmp;
		}
		for(size_t col = 0; col < cols; ++col) {
			std::cout << m[row][col] << " ";
		}
		std::cout << std::endl;
	}
	for(int row = 0; row < rows; ++row)
		delete[] m[row];
	delete[] m;
}

Cpp-style решение (как надо):

#include <iostream>
#include <vector>
#include <iterator>
#include <algorithm>
void invert_cols(size_t rows, size_t cols, int n) {
	std::vector<std::vector<int>> m(rows, std::vector<int>(cols));
	std::for_each(m.begin(), m.end(), [](std::vector<int>& row) { 
		std::iota(row.begin(), row.end(), n); 
		std::reverse(row.begin(), row.end()); 
		std::copy(row.begin(), row.end(), std::ostream_iterator<int>(std::cout, " "));
		std::cout << std::endl;
	});
}

Answer 1

[jcmvbkbc]

Почему программа ломается при освобождении памяти?

Потому что во время работы кто-то вылез за пределы выделенной памяти и перезаписал служебную информацию в куче. Простейший способ отладить это -- запуском программы под valgrind.
Но в этом вопросе всё проще:

char *response = new char[line.size()];
  std::strcpy(response, line.c_str());

Должно быть char *response = new char[line.size() + 1];

Comments

[Владимир Олохтонов]

Поскольку терминирующий ноль.

Answer 2

[Vitaly]

Для начала, я бы убрал явное использование new/delete. Это всё-таки C++. У вас есть класс std::string, его и используйте везде (например при возврате значений). Этот класс предоставляет метод reserve, чтобы выделить необходимое количество памяти. У него есть метод data, если необходимо передать в C API char*.

Вполне возможно, что у вас не крэш, а программа просто висит на send/post, ожидая ответа. Это тоже стоит проверить.