Вылет при удалении динамического массива?

Question

[Имя]

Когда удаляю - исключение.
Почему? объясните с максимальной подробностью.

Код:

Data parseAnswerFromCOM(const char* source)
{
	Data data = { };
	char *s = new char[strlen(source) + 1];
	char *next_token1 = NULL;
	strcpy_s(s, strlen(s), source);
	char *p = strtok_s(s, ";", &next_token1);
	int tmpCounter = 0;
	while (p != NULL)
	{
		cout << p << endl;
		p = strtok_s(NULL, ";", &next_token1);
	}
	delete[] s;
	return data;
}

Comments

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

Герман , исключение в студию!
Кстати, место генерации исключения тоже в студию!

Если подумать, то strcpy_s(s, strlen(s), source); - это прекрасный способ сделать ультраненадежный вызов "надежной" функции.

[Roman]

Почему? объясните с максимальной подробностью.

Надо std::string

#include <iostream>
#include <string>

void parseAnswerFromCOM(const std::string& source)
{
  std::string::size_type first = 0;
  std::string::size_type last  = 0;

  while((last = source.find(';', first)) != std::string::npos)
  {
    std::string r = source.substr(first, last - first);
    std::cout << r << "\n";
    first = last + 1;
  }
}

int main()
{
  std::string s = "asdf;ghjkl;vbnhg;poiuy;";
  parseAnswerFromCOM(s);
  std::cin.get();
}

When you use the C-style string functions with Microsoft Visual
Studio, the compiler is likely to give you security-related warnings or even
errors about these functions being deprecated. You can eliminate these warnings
by using other C Standard Library functions, such as strcpy_s() or
strcat_s(), which are part of the “secure C library” standard (ISO/IEC TR
24731). However, the best solution is to switch to the C++ Standard Library
std::string class.

Answer 1

[jcmvbkbc]

char *s = new char[strlen(source) + 1];
...
strcpy_s(s, strlen(s), source);

strlen(s) -- вообще за гранью добра и зла и возвращает случайное число, по большому счёту. Потому что s -- только что выделенный неинициализированный массив.

Вызов strcpy_s совершенно бессмысленный и неправильный. Бессмысленный, потому что ты только что выделил столько памяти, сколько данных в строке. strcpy_s имеет смысл, если буфер в который ты копируешь -- это массив фиксированной длины. Неправильный, потому что даже если ты имел в виду strlen(source), а не strlen(s), то должен был передать strlen(source) + 1, чтобы было куда 0-терминатор скопировать. Кроме всего прочего, strcpy_s ещё и возвращает код ошибки, который имеет смысл проверить.

Лучше всего в этом месте было бы вызвать memcpy(s, source, strlen(source) + 1); или вообще выкинуть s = new char [...]; strcpy_s ... delete [] s; и заменить на s = strdup(source); ... free(s);
Если очень хочется strcpy_s, то хотя бы так: strcpy_s(s, strlen(source) + 1, source);

Исправление этого места починит и delete, потому что delete валится из-за того, что ты поломал heap.

Comments

[res2001]

вообще за гранью добра и зла

+5

Герман, strdup() - не переносимо, т.к. это POSIX. В винде ее нет.