huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Clean up Header qvalue parsing

This commit is contained in:
John Luo 2017-03-06 14:28:04 -08:00
parent 7440d5d29c
commit a1f4928ed5
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Microsoft.Net.Http.Headers/HeaderUtilities.cs
View File

@ -13,7 +13,7 @@ namespace Microsoft.Net.Http.Headers
public static class HeaderUtilities public static class HeaderUtilities
{ {
private static readonly int _int64MaxStringLength = 19; private static readonly int _int64MaxStringLength = 19;
private static readonly int _qualityValueMaxCharCount = 10; // Little bit more permissive than RFC7231 5.3.1 private static readonly int _qualityValueMaxCharCount = 10; // Little bit more permissive than RFC7231 5.3.1
private const string QualityName = "q"; private const string QualityName = "q";
internal const string BytesUnit = "bytes"; internal const string BytesUnit = "bytes";
@ -503,7 +503,7 @@ namespace Microsoft.Net.Http.Headers
var ch = input[current]; var ch = input[current];
if (ch >= '0' && ch <= '1') // Only values between 0 and 1 are accepted, according to RFC if (ch >= '0' && ch <= '1') // Only values between 0 and 1 are accepted, according to RFC
{ {
intPart = ch - '0'; intPart = ch - '0';
current++; current++;
@ -560,8 +560,10 @@ namespace Microsoft.Net.Http.Headers
quality = intPart; quality = intPart;
} }
if (quality < 0 || quality > 1) if (quality > 1)
{ {
// reset quality
quality = 0;
return false; return false;
} }