fopen не работает с GetLastError() = 0 - почему?

Question

У меня есть загруженная среда с большим количеством потоков, открывающих один и тот же файл для чтения. проблема - я получаю null, когда я fopen, но errno равен 0.

Как это возможно?

Код:

if ((fParm = fopen(FullFileName.c_str(), "r")) == NULL) 
    { 
     printf("%d", GetLastError()); 
    } 

Как указано в MSDN - после того, как Еореп терпит неудачу - вы должны проверить ERRNO. и это то, что мы делаем:

http://msdn.microsoft.com/en-us/library/yeby3zcb.aspx

(игнорировать Printf, он не используется в исходном коде - я написал только упрощать вопрос).

Благодарим за помощь.

---------------------- РЕДАКТИРОВАТЬ ----------------------- ---

я фактически использовал GetLastError() вместо старого доброго»ERRNO - который вызвал ошибку, чтобы показать 0.

Answer 1

Если это Windows и вы используете GetLastError, ваш код не будет работать. Документация ясна:

Каждая из этих функций [fopen и _wfopen] возвращает указатель на открытый файл. Значение нулевого указателя указывает на ошибку. Если имя файла или режим NULL или пустая строка, эти функции запускают недействительный обработчик параметров, который описан в разделе «Проверка параметров». Если выполнение разрешено продолжить, эти функции возвращают NULL и устанавливают errno в EINVAL. - (подчеркивание добавлено) MSDN

Как и на других платформах, fopen возвращает ошибку, установив errno. Вы не должны звонить GetLastError, но регистрируете значение errno.Это две совершенно разные вещи.

Вызов GetLastError может быть случайным в некоторых случаях. Если основной причиной ошибки является ошибка другой функции в Windows API (например, CreateFile), возможно, что другая функция может вызвать SetLastError, и вполне возможно, что никакой другой код в fopen не изменит это значение, поэтому иногда это может произойти для правильной ошибки. Но если ошибка исходит от fopen, то GetLastError не даст правильной информации.

Answer 2

стандарт C не гарантирует, что errno будет установлен содержательного значения, хот POSIX Определяет что он вернет NULL в случае отказа и установит errno, IEEE Std 1003.1-2013.

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

Вы упомянули в комментариях, что вы использовали GetLastError() на Windows, который должен быть thread-safe. К сожалению, вы не отредактировали главный вопрос, чтобы отразить все эти факты, которые разбросаны в комментариях.

Answer 3

Мы должны угадать, потому что вы не показали нам свой настоящий код. Но одна возможность: ваш реальный код делает что-то вроде этого:

if ((fParm = fopen(FullFileName.c_str(), "r")) == NULL) 
{ 
     SomeLogMacro(Severity, "%d", errno); 
} 

, расширяющий макро-то вроде этого:

if ((fParm = fopen(FullFileName.c_str(), "r")) == NULL) 
{ 
     do { if (ShouldLog(Severity)) DoLog("%d", errno); } while (false); 
} 

Если функция ShouldLog изменяет значение errno, вы будете регистрировать неверное значение ,

Answer 4

Чтение MSDN documentation for fopen Я вижу, что он не устанавливает lasterror (он просто возвращает 0 при ошибке). Чтение MSDN documentation for GetLastError Я вижу, что он возвращает только, ну, последнюю ошибку, когда-либо установленную кем-либо (не fopen).

Answer 5

Редактировать Дэвид Шварц был прав: проблема заключается в том, что Еореп не вызывает GetLastError - смотрите ниже

Хорошо, я увидел в комментариях к другим ответам, которые вы имеете в #define GetLastError() errno. Вы должны были указать, что в начальном вопросе, поскольку errno должен быть потокобезопасным Posix, но я не уверен, что он находится в Windows, когда я вижу это предупреждение в MSDN о глобальных переменных (включая errno):

Эти глобальные переменные являются нежелательными для более безопасных функциональных версий, которые должны быть использованы вместо глобальной переменной

Но GetLastErrorявляется поточно, просто Microsoft говорит:

Вы должны сразу вызвать функцию GetLastError, когда возвращаемое значение функции указывает, что такой вызов вернет полезные данные. Это связано с тем, что некоторые функции вызывают SetLastError с нулевым значением, когда они преуспевают, уничтожая код ошибки, установленный самой последней неудачной функцией.

Так что единственный надежный способ, чтобы иметь переменную локального, скажем lastError и сделать:

if ((fParm = fopen(FullFileName.c_str(), "r")) == NULL) 
    { 
     int lastError = errno; // in reality calls GetLastError immediately and saves it current value even if other functions later calls SetLastError(0) 
     printf("%d", lastError); 
    } 

EDIT:

В самом деле, fopen не вызывает SetLastError сам по себе. Вот демо проблемы:

#include <stdio.h> 
#include <windows.h> 
#include <errno.h> 

int main() { 
    FILE *fd = fopen("", "r"); 
    int lastErr = GetLastError(); 
    int err = errno; 

    printf("errno : %d - GetLastError %d\n", err, lastErr); 
    return 0; 
} 

и результат:

errno : 22 - GetLastError 0 

В самом деле, что виноват в #define GetLastError() errno. Но в многопоточном приложении я настоятельно рекомендую немедленно скопировать errno в локальную переменную.