stat() behavior differs on Win9x and WinNT

[Chris Faylor]

On Mon, May 08, 2000 at 11:51:53AM +0200, Corinna Vinschen wrote:
>"Fifer, Eric" wrote:
>> 
>> With recent snapshots, stat() of a file with an invalid name,
>> like stat("abc>def") ('>' is not allowed in file names), does
>> not fail on Win98:
>> [...]
>>   Win98:
>>     symlink_info::check: GetFileAttributesA (c:\efifer\opt\abc>def) failed
>>     seterrno: 161 (BAD_PATHNAME) -> 22
>>     [...]
>> 
>>   WinNT:
>>     symlink_info::check: GetFileAttributesA (c:\efifer\opt\abc>def) failed
>>     seterrno: 123 (INVALID_NAME) -> 2
>> 
>> A patch is trival, but I'm not sure which is correct:
>> 
>>   + map BAD_PATHNAME to ENOENT (instead of the current EINVAL), but
>>     I have no idea how widespread the impact of this might be.  Or,
>>     maybe INVALID_NAME should really be EINVAL?
>> 
>>   + change the test in stat_worker to:
>> 
>>       (!oret && get_errno () != ENOENT && get_errno () != EINVAL)
>
>IMO the mapping ERROR_BAD_PATHNAME -> EINVAL is incorrect.

I agree.  Is the closest UNIX equivalent ENOENT?  If so, let's make this
change.

cgf