Skip to content

Commit

Permalink
Don't recheck for attributes if the path is too long (dotnet/corefx#3…
Browse files Browse the repository at this point in the history
…9299)

One of our existing tests hits this on some machines. There is no point in retrying if the path is too long and we don't want to add it to the assert either.

Fixes dotnet/corefx#39048

Commit migrated from dotnet/corefx@af2e995
  • Loading branch information
JeremyKuhne authored Jul 9, 2019
1 parent cdf3c8a commit d336918
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -217,7 +217,8 @@ internal static int FillAttributeInfo(string path, ref Interop.Kernel32.WIN32_FI
&& errorCode != Interop.Errors.ERROR_INVALID_PARAMETER
&& errorCode != Interop.Errors.ERROR_NETWORK_UNREACHABLE
&& errorCode != Interop.Errors.ERROR_NETWORK_ACCESS_DENIED
&& errorCode != Interop.Errors.ERROR_INVALID_HANDLE // eg from \\.\CON
&& errorCode != Interop.Errors.ERROR_INVALID_HANDLE // eg from \\.\CON
&& errorCode != Interop.Errors.ERROR_FILENAME_EXCED_RANGE // Path is too long
)
{
// Assert so we can track down other cases (if any) to add to our test suite
Expand Down

0 comments on commit d336918

Please sign in to comment.