diff options
author | Paulo Gomes <pjbgf@linux.com> | 2023-10-09 21:58:18 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-10-09 21:58:18 +0100 |
commit | 771a3eb05b2937496910bc2c2cf4c61ccb5df010 (patch) | |
tree | 69e25be54ee6e3405ba299a877c11373ae26b407 /plumbing/protocol/packp/srvresp.go | |
parent | 7ef7dc7e3895a01f7149fff6238bd338ca16949d (diff) | |
parent | 1d26511c717ea334a16499a018d2b877b557be79 (diff) | |
download | go-git-771a3eb05b2937496910bc2c2cf4c61ccb5df010.tar.gz |
Merge pull request #868 from pjbgf/fix-fuzz
plumbing: protocol/packp, Add validation for decodeLine
Diffstat (limited to 'plumbing/protocol/packp/srvresp.go')
-rw-r--r-- | plumbing/protocol/packp/srvresp.go | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/plumbing/protocol/packp/srvresp.go b/plumbing/protocol/packp/srvresp.go index 8cd0a72..a9ddb53 100644 --- a/plumbing/protocol/packp/srvresp.go +++ b/plumbing/protocol/packp/srvresp.go @@ -101,12 +101,14 @@ func (r *ServerResponse) decodeLine(line []byte) error { return fmt.Errorf("unexpected flush") } - if bytes.Equal(line[0:3], ack) { - return r.decodeACKLine(line) - } + if len(line) >= 3 { + if bytes.Equal(line[0:3], ack) { + return r.decodeACKLine(line) + } - if bytes.Equal(line[0:3], nak) { - return nil + if bytes.Equal(line[0:3], nak) { + return nil + } } return fmt.Errorf("unexpected content %q", string(line)) |