Почему в моем коде портится содержимое переменных?

Question

[gaki]

Вложенная функция fopen_s принимает указатели в параметрах функции, после чего коверкает преданные указатели и обрабатывает.

Код функции.

int fileRead ( char *file_name )
{
	FILE *stream;

	if ( fopen_s ( &stream, file_name, "r" ) != 0 )
		return ENOENT;

	char buffer[char_buffer_size];
	while ( ( fgets ( buffer, char_buffer_size, stream ) ) != NULL )
		printf ( "\n%s", buffer );

	if ( fclose ( stream ) != 0 )
		return ENOENT;

	return 0;
}

Запуск функции с ошибкой.

if ( fileRead ( file_name ) != 0 )
		printf ( "\nОшибка вывода файла." );

Момент запуска функции fileRead.

Момент выхода из функции fopen_s.

Запуск функции без ошибки.

if ( fileRead ( "ff.routes" ) != 0 )
		printf ( "\nОшибка вывода файла." );

Момент запуска функции fileRead.

Момент выхода из функции fopen_s.

Comments

[lz961]

как определена переменная file_name, которая передаётся в функцию?

[gaki]

lz961,
Сама функция.

char *user_getStr ( char *print )
{
	if ( print != "0" )
		printf ( print );

	char buffer[char_buffer_size];

	if ( fgets ( buffer, char_buffer_size, stdin ) != NULL )
	{
		if ( buffer == "\n" )
			return NULL;

		char *str = charEndStr ( buffer );
		return str;
	}
	else
	{
		dirent_set_errno ( EIO );
		return NULL;
	}
}

Момент получения значения из функции.

if ( ( file_name = user_getStr ( "\nНазвание файла: " ) ) == NULL )
	{
		printf ( "\nНеверное значение." );
		return;
	}

[lz961]

gaki, charEndStr -- что за зверь?

[gaki]

lz961, меняет последний элемент строки. В данном случае "\n" на "\0"

[lz961]

gaki, вам уже ответили на вопрос. Если функция charEndStr не создаёт новой строки в куче, а просто модифицирует переданную ей строку то указатель str у вас указывает на нулевой элемент локального массива buffer время жизни которого -- время исполнения функции user_getStr.
плохое решение решение -- создавать буфер в user_getStr в куче:

char *user_getStr ( char *print )
{
  if ( print != "0" )
    printf ( print );

  char * buffer = calloc(char_buffer_size, 1); // изменено здесь

  if ( fgets ( buffer, char_buffer_size, stdin ) != NULL )
  {
    if ( buffer == "\n" )
      return NULL;

    char *str = charEndStr ( buffer );
    return str;
  }
  else
  {
    dirent_set_errno ( EIO );
    return NULL;
  }
}

[gaki]

lz961, Благодарю.

Answer 1

[Wataru]

Проблема в переменной file_name, а в не функции. Судя по адресу, она на стеке. Причем на месте где-то в середине лакальной переменной buffer.

Это значит, что вы из какой-то функции раньше вернули указатель на локальную переменную и получили висячий указатель.

Comments

[gaki]

Спасибо за помощь.