Почему вылетает исключение в MSVC?

Question

[evg_96]

В чем проблема или как это можно пофиксить? В gcc все отпрабатывает нормально

#define _CRT_SECURE_NO_WARNINGS

#include <stdio.h>
#include <conio.h>
#include <string.h>
#include <stdlib.h>

#define TSIZE 45

struct film
{
    char title[TSIZE];
    int rating;
    struct film * next;
};

char * s_gets(char * str, int n);

int main(int argc, char * argv[])
{
    struct film * head = NULL;
    struct film * prev, * current;
    char input[TSIZE];

    puts("Input first name of movie: ");

    while (s_gets(input, TSIZE) != NULL && input[0] != '\0')
    {
        current = (struct film *) malloc(sizeof(struct film));

        if (head == NULL)
            head = current;
        else
            prev->next = current;

        current->next = NULL;

        strcpy(current->title, input);

        puts("Input your rating: ");
        scanf("%d", &current->rating);

        while (getchar() != '\n')
            continue;

        puts("Input the next name of movie: ");

        prev = current;
    }

    if (head == NULL)
        printf("Data not found");
    else
        printf("List of movies:\n");

    current = head;

    while (current != NULL)
    {
        printf("Movie: %s, rating: %d\n", current->title, current->rating);

        current = current->next;
    }

    current = head;

    while (current != NULL)
    {
        free(current);

        current = current->next;
    }

    printf("The programm is completed.\n");

    _getch();

    return 0;
}

char * s_gets(char * str, int n)
{
    char * value;
    char * find;

    value = fgets(str, n, stdin);

    if (value)
    {
        find = strchr(str, '\n');

        if (find)
            *find = '\0';
        else
            while (getchar() != '\n')
                continue;
    }

    return value;
}

Comments

[res2001]

Запустите под отладчиком, там ошибка находится в течении 5 минут.

[evg_96]

res2001, не могу я определить в чем ошибка и как пофиксить)

[res2001]

evg_96, Почему?
Обычно я делаю так: собираю приложение с отладочной информацией, запускаю под отладчиком в IDE, стартую без точек останова. Там где произойдет ошибка программа остановится и вернется в отладчик, в отладчике нужно по стеку вызовов перейти на свою последнюю функцию и уже в нужной точке анализируешь, что ж ты там написал.

Иногда бывает, что программа с отладочной информацией работает, а в "релизной" сборке не работает - значит где-то выход за пределы массива/обращение к не выделенной памяти (не инициализированному указателю)...

На каком этапе работы программа вываливается (это можно определить по сообщениям, которые выводятся программой)? Какие входные данные подаете при этом? На чем именно из входных данных программа вываливается?

[evg_96]

res2001, у меня получается что компилятор при компиляции предупреждает что функции strcpy и scanf небезопасны. В общем то программа запускается, ввожу данные, сначала строковое значение, потом числовое в цикле. После того как будет введена "пустая строка", программа должна вывести список введенных данных. Данные впринципе выводятся, то также вылетает и исключение которое показывал выше. И вот не могу понять в чем проблема. Да и в gcc все нормально отрабатывает, а вот конкретно в MSVC не все так гладко

[res2001]

evg_96, Ошибка тут:

while (current != NULL)
    {
        free(current);
        current = current->next;
    }

Думаю догадаетесь сами где конкретно.

[evg_96]

res2001, я не понимаю(((( по моим скиллам для меня тут все нормально, ну никак не могу разобраться...
что все так трудно то?!!!

[res2001]

evg_96, Вы сначала освободили память: free(current), а потом обращаетесь к уже освобожденной памяти: current->next. После free указатель уже не действительный и его использовать нельзя.
В gcc ошибка не проявляется, потому что используются разные менеджеры памяти внутри стандартной библиотеки (она хоть и стандартная снаружи, но внутри все реализовывают по разному).
Вообще, по моему, тут и компилятор мог ругнуться, возможно надо выставить уровень предупреждений по выше.

Answer 1

[Александр Ананьев]

Программа падает тут
free(current);
Падает, потому что после освобождения памяти у current вы пытаетесь потом пользоваться этой памятью. Соответственно возникает Access Violation/

Рабочий вариант кода

#define _CRT_SECURE_NO_WARNINGS

#include <stdio.h>
#include <conio.h>
#include <string.h>
#include <stdlib.h>

#define TSIZE 45

struct film
{
	char title[TSIZE];
	int rating;
	struct film * next;
};

char * s_gets(char * str, int n);

int main(int argc, char * argv[])
{
	struct film * head = NULL;
	struct film * prev, *current;
	char input[TSIZE];

	puts("Input first name of movie: ");

	while (s_gets(input, TSIZE) != NULL && input[0] != '\0')
	{
		current = (struct film *) malloc(sizeof(struct film));

		if (head == NULL)
			head = current;
		else
			prev->next = current;

		current->next = NULL;

		strcpy(current->title, input);

		puts("Input your rating: ");
		scanf("%d", &current->rating);

		while (getchar() != '\n')
			continue;

		puts("Input the next name of movie: ");

		prev = current;
	}

	if (head == NULL)
		printf("Data not found");
	else
		printf("List of movies:\n");

	current = head;

	while (current != NULL)
	{
		printf("Movie: %s, rating: %d\n", current->title, current->rating);

		current = current->next;
	}

	current = head;

	while (current != NULL)
	{
		auto next = current->next;
		free(current);


		current = next;
	}

	printf("The programm is completed.\n");

	_getch();

	return 0;
}

char * s_gets(char * str, int n)
{
	char * value;
	char * find;

	value = fgets(str, n, stdin);

	if (value)
	{
		find = strchr(str, '\n');

		if (find)
			*find = '\0';
		else
			while (getchar() != '\n')
				continue;
	}

	return value;
}

Comments

[evg_96]

Александр Ананьев Все равно вылетает исключение

PS. struct film * next = current->next; Так работает.

[Mercury13]

evg_96, Этот товарищ не стал заморочиваться с типами, и добавил фичу из Си++11. Хотя на вид тут чистый Си.