Fix cookie null regression #45665 (#45715)

Author: Tratcher

src/Http/Http/test/RequestCookiesCollectionTests.cs

@@ -44,4 +44,29 @@ public void ParseManyCookies()
 
         Assert.Equal(12, cookies.Count);
     }
+
+    [Theory]
+    [InlineData(",", null)]
+    [InlineData(";", null)]
+    [InlineData("er=dd,cc,bb", new[] { "dd" })]
+    [InlineData("er=dd,err=cc,errr=bb", new[] { "dd", "cc", "bb" })]
+    [InlineData("errorcookie=dd,:(\"sa;", new[] { "dd" })]
+    [InlineData("s;", null)]
+    public void ParseInvalidCookies(string cookieToParse, string[] expectedCookieValues)
+    {
+        var cookies = RequestCookieCollection.Parse(new StringValues(new[] { cookieToParse }));
+
+        if (expectedCookieValues == null)
+        {
+            Assert.Equal(0, cookies.Count);
+            return;
+        }
+
+        Assert.Equal(expectedCookieValues.Length, cookies.Count);
+        for (int i = 0; i < expectedCookieValues.Length; i++)
+        {
+            var value = expectedCookieValues[i];
+            Assert.Equal(value, cookies.ElementAt(i).Value);
+        }
+    }
 }

src/Http/Shared/CookieHeaderParserShared.cs

@@ -29,9 +29,12 @@ public static bool TryParseValues(StringValues values, IDictionary<string, strin
                 if (TryParseValue(value, ref index, supportsMultipleValues, out var parsedName, out var parsedValue))
                 {
                     // The entry may not contain an actual value, like " , "
-                    var name = enableCookieNameEncoding ? Uri.UnescapeDataString(parsedName.Value.Value!) : parsedName.Value.Value!;
-                    store[name] = Uri.UnescapeDataString(parsedValue.Value.Value!);
-                    hasFoundValue = true;
+                    if (parsedName != null && parsedValue != null)
+                    {
+                        var name = enableCookieNameEncoding ? Uri.UnescapeDataString(parsedName.Value.Value!) : parsedName.Value.Value!;
+                        store[name] = Uri.UnescapeDataString(parsedValue.Value.Value!);
+                        hasFoundValue = true;
+                    }
                 }
                 else
                 {