"ошибка сегментирования" как исправить(С++)?

Question

[Молзв Реках]

Всем привет! Пишу мульти-программу с функционалом которая упрощает жизнь, там присутствует проверка файла на вирусы, где я копирую код файла, и сравниваю с файлами которые лежат в моей папке с БД вирусами(пока что только один). В итоге во время копирования у меня выскакивает ошибка. Что делать? Вот код:

void antivirus(string path_file) {
	ifstream f;
	f.open(path_file);
	char *str;
	while (!f.eof())
	{
		char *str=new char[255];
		f.getline(str,255);
	}
	f.close();

	ifstream virus;
	virus.open("viruses/virus1.cs");
	char *vir;
	while (!virus.eof())
	{
		char *vir = new char[255];
		virus.getline(str,255);
	}
	virus.close();

	cout << "Проверка прошла успешно." << endl;	

	if (str == vir) {
		cout << "Файл вредоносный!" << endl;
	}
	else {
		cout << "Файл безопасен!" << endl;
	}
}

Я никогда не встречался с такой ошибкой, не знаю как исправить и что вообще делать. Программа кроссплатформенная, и в начале определяет систему(не знаю зачем я это сделал, ведь на плюсах нужно во время написания определять под какую платформу я буду писать, но всё же), но не в этом суть. Строчек кода: 400.

Comments

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

SaddledSharp , так а с какой ты все таки ошибкой столкнулся?
Приведи саму ошибку, укажи в своем коде место, на котором у тебя останавливается отладчик.

char *str;
while (!f.eof())
 {
   char *str=new char[255];
   f.getline(str,255);
 }
 f.close();

Ну это так не пойдет. И с vir тоже, кстати. У тебя просто память выбрасывается в никуда - утекает.
Еще у тебя не видно использование delete[] для str и vir.

if (str == vir) {
Ну это так тоже не пойдет. Такое условие будет истинным только когда str и vir будут равны nullptr, а этого у тебя в коде нет, значит истинным такое условие не станет никогда.

[Молзв Реках]

Евгений Шатунов, Мда... Я учусь на практике, и решил такую идею реализовать в реальность. Я говорю, я ничего не понял, на протяжении всех 500 строчек кода, я 200 из них нагуглил. Окей, я добавил delete[] для vir и str но дальше я что то ничего не понял... Прошу помощи... Исправьте код.

[Roman]

Области видимости...
str и vir uninitialized

[Молзв Реках]

Roman, Исправил, программа работает, но после выполнения выскакивает вторая ошибка: "double free or corruption (out)". И да, я немного не понял как сравнивать файлы(текст в них)? Подскажите, пожалуйста.

Answer 1

[lorc]

В данном случае ошибка в строке virus.getline(str,255);. Там должно быть virus.getline(vir,255);. Именно это приводит к segfault. Но как я написал в комментарии к другому ответу - это не единственная проблема в вашем коде.

Comments

[Молзв Реках]

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

[lorc]

SaddledSharp, к сожалению, так оно тоже не будет работать. Вы кроме всего прочего пытаетесь делать delete[], хотя не делали new[] в этом скоупе. Вообще, вы несколько раз усложнили решение.

На данном этапе вы пытаетесь сравнить два файла. Представьте что файлы по размеру больше, чем доступная оперативная память? Как быть в таком случае? Логично бы читать в одном цикле сразу оба файла, по кускам (ну например по тех же 255 байт) и сравнивать эти куски. Таким образом, вам не надо вычитывать целиком оба файла в память. Потом, вам нужно разобраться с понятием "указатель" и понять почему virt == str будет всегда false (в вашем коде).

Я бы сделал как-то так:

ifstream f1;
ifstream f2;
f1.open("viruses/virus1.cs");
f2.open("file2");
bool result = true;

char buf1[256];
char buf2[256];

while ( ! f1.eof() && !f2.oef() )
{
  f1.read(buf1, 256);
  f2.read(buf2, 256);
  if (f1.gcount() != f2.gcount())
  {
    result = false;
    break;
  }

  if (memcmp(buf1, buf2, f1.gcount()) !=0)
  {
    result = false;
    break;
  }
}

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

[Молзв Реках]

lorc, Спасибо большое!!!! Всё отлично работает. Буду дальше писать программу.

Answer 2

[Александр]

Ошибка связанна с памятью. Си язык, который требует точного описания действий, которое вы поленились сделать. Где выделение памяти для переменной? Компилятор не будет угадывать сколько нужно памяти для конкретного файла.

Comments

[Молзв Реках]

Блин, можете указать именно строчку кода где я это упустил? Я просто на С++ только неделю учусь.

[lorc]

К сожалению там множество ошибок. Как-то даже тяжело все перечислить.

1. Вы внутри цикла заводите переменную с тем же именем, что уже была объявлена выше
2. На каждой итерации цикла вы выделяете новый буфер размером 255 байт, а старый - не освобождаете. В результате у вас утекает память.
3. Вы читаете файл по строкам. У вас там текстовые файлы? Или двоичные?
4. В самом конце вы сравниваете указатели, а не содержимое буферов
5. Из-за ошибки copy-pastе вы во втором цикле тоже читаете в str, который просто не инициализирован. Именно это и приводит к segfault, кстати.

Вам нужно исправить все ошибки, иначе программа просто не будет работать правильно

[Молзв Реках]

lorc, Некоторые ошибки мне как то удалось понять и исправить. Боюсь запускать програму, в прошлый раз комп просто завис(за это я люблю этот язык :D). Вот код, будет работать?:

void antivirus(string path_file) {
	ifstream f;
	f.open(path_file);
	char str[512];
	while (!f.eof())
	{
		char *str=new char[255];
		f.getline(str,255);
	}
	f.close();

	ifstream virus;
	virus.open("viruses/virus1.cs");
	char vir[512];
	while (!virus.eof())
	{
		char *vir = new char[255];
		virus.getline(vir,255);
	}
	virus.close();

	cout << "Проверка прошла успешно." << endl;	

	if (str == vir) {
		cout << "Файл вредоносный!" << endl;
	}
	else {
		cout << "Файл безопасен!" << endl;
	}
	delete[] str;
	delete[] vir;
}

Добавлено: я немного не понял как сравнивать эти файлы. И да, это не txt файлы, я даже не знаю как сказать, ведь я изначально не продумывал ТОЛЬКО на txt, по моей логике вирус может быть как и на питоне, так и на С++, так и на шарпе. Но в основном это открытые файлы и их можно прочитать в редакторе, так что по идее, это txt с расширение .cs, .py, .cpp