Archives/Cloak
2
0
Fork 0
mirror of https://github.com/cbeuw/Cloak.git synced 2024-11-15 18:13:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use normal Read in ReadTLS instead of ReadFull

Browse Source
This commit is contained in:
Andy Wang 2019-08-30 14:38:31 +01:00
parent 4fb1f55e2d
commit 59f2fe04e3
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
internal/util/util.go
View File

@ -58,15 +58,13 @@ func ReadTLS(conn net.Conn, buffer []byte) (n int, err error) {
left := dataLength
readPtr := 5
// TODO: Deadline here?
for left != 0 {
// If left > buffer size (i.e. our message got segmented), the entire MTU is read
// if left = buffer size, the entire buffer is all there left to read
// if left < buffer size (i.e. multiple messages came together),
// only the message we want is read
// TODO: Why ReadFull here? Shouldn't it be just normal read since we adjust left and readPtr according to read amount?
i, err = io.ReadFull(conn, buffer[readPtr:readPtr+left])
i, err = conn.Read(buffer[readPtr : readPtr+left])
if err != nil {
return
}