Use IndexOfAny in HttpListener.HandleAuthentication (#71137)

* Use IndexOfAny in HttpListener.HandleAuthentication Just cleaning up a manual loop that can instead be IndexOfAny. * Fix missing check
dotnet · Jun 23, 2022 · 982946a · 982946a
1 parent 70e9ca0
commit 982946a
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/libraries/System.Net.HttpListener/src/System/Net/Windows/HttpListener.Windows.cs b/src/libraries/System.Net.HttpListener/src/System/Net/Windows/HttpListener.Windows.cs
@@ -803,13 +803,10 @@ public HttpListenerContext EndGetContext(IAsyncResult asyncResult)
                 if (authorizationHeader != null && (authenticationScheme & ~AuthenticationSchemes.Anonymous) != AuthenticationSchemes.None)
                 {
                     // Find the end of the scheme name.  Trust that HTTP.SYS parsed out just our header ok.
-                    for (index = 0; index < authorizationHeader.Length; index++)
+                    index = authorizationHeader.AsSpan().IndexOfAny(" \t\r\n");
+                    if (index < 0)
                     {
-                        if (authorizationHeader[index] == ' ' || authorizationHeader[index] == '\t' ||
-                            authorizationHeader[index] == '\r' || authorizationHeader[index] == '\n')
-                        {
-                            break;
-                        }
+                        index = authorizationHeader.Length;
                     }
 
                     // Currently only allow one Authorization scheme/header per request.