Merge pull request #41 from justcoding121/master

#36 fix extension exist check
justcoding121 · Oct 11, 2018 · be2094d · be2094d
2 parents edcaa90 + b175578
commit be2094d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/StreamExtended/SslTools.cs b/src/StreamExtended/SslTools.cs
@@ -173,7 +173,7 @@ public static async Task<bool> IsClientHello(CustomBufferedStream stream, IBuffe
 
                 Dictionary<string, SslExtension> extensions = null;
 
-                if(extenstionsStartPosition + 5 < recordLength)
+                if(extenstionsStartPosition < recordLength + 5)
                 {
                     extensions = await ReadExtensions(majorVersion, minorVersion, peekStream, bufferPool, cancellationToken);
                 }
@@ -326,7 +326,7 @@ public static async Task<bool> IsServerHello(CustomBufferedStream stream, IBuffe
 
                 Dictionary<string, SslExtension> extensions = null;
 
-                if (extenstionsStartPosition + 5 < recordLength)
+                if (extenstionsStartPosition < recordLength + 5)
                 {
                    extensions = await ReadExtensions(majorVersion, minorVersion, peekStream, bufferPool, cancellationToken);
                 }