On 06/29/15 18:46, YongHaoHu wrote:
On 15/6/29 下午10:54, Piotr Caban wrote:
On 06/29/15 16:42, YongHaoHu wrote:
Why are you handling ERROR_BAD_DEV_TYPE error here?
I don't have special reason, just base on the test result on testbot.
It looks wrong. Could you please send the test here and point on what testbot machine it happens?
Yes, it's wrong. According to your advice, I tested GetFileAttributesA and found that its last error handling was also not correct. (It means that many ntdll functions have problems with last_error handling.) The code I used is : GetFileAttributesA("\\.\PiPe\tests_pipe.c"); printf("getlast err is %d\n", GetLastError()); The output in wine is "getlast err is 66" while "|getlast err is 0" in testbot.
Here is the test result: https://testbot.winehq.org/JobDetails.pl?Key=14832&log_101=1#k101 Plesase ignore the failed msg.
In this case test should be marked with todo_wine and fail in wine. Other option is to fix GetFileAttributes function.