From 4a4879ea37677babf224ea055bb69b7371f1bc42 Mon Sep 17 00:00:00 2001 From: Qian Wang Date: Sat, 15 Jun 2019 11:56:52 +1000 Subject: [PATCH] Correct the maximum record size --- cmd/ck-client/ck-client.go | 6 +++--- cmd/ck-server/ck-server.go | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/cmd/ck-client/ck-client.go b/cmd/ck-client/ck-client.go index 99c2e02..b76c5af 100644 --- a/cmd/ck-client/ck-client.go +++ b/cmd/ck-client/ck-client.go @@ -21,11 +21,11 @@ import ( var version string -//TODO: take iv into account func pipe(dst io.ReadWriteCloser, src io.ReadWriteCloser) { - // The maximum size of TLS message will be 16396+12. 12 because of the stream header + // The maximum size of TLS message will be 16380+12+16. 12 because of the stream header and 16 + // because of the salt/mac // 16408 is the max TLS message size on Firefox - buf := make([]byte, 16396) + buf := make([]byte, 16380) for { i, err := io.ReadAtLeast(src, buf, 1) if err != nil { diff --git a/cmd/ck-server/ck-server.go b/cmd/ck-server/ck-server.go index 0205256..b4fa737 100644 --- a/cmd/ck-server/ck-server.go +++ b/cmd/ck-server/ck-server.go @@ -20,11 +20,11 @@ import ( var version string -//TODO: take iv into account func pipe(dst io.ReadWriteCloser, src io.ReadWriteCloser) { - // The maximum size of TLS message will be 16396+12. 12 because of the stream header + // The maximum size of TLS message will be 16380+12+16. 12 because of the stream header and 16 + // because of the salt/mac // 16408 is the max TLS message size on Firefox - buf := make([]byte, 16396) + buf := make([]byte, 16380) for { i, err := io.ReadAtLeast(src, buf, 1) if err != nil {