prog: fix out-of-bounds access

ParseLog can access data out-of-bounds.
Fix that and fix regression fuzz tests to catch this.

3 files changed, 6 insertions, 4 deletions

diff --git a/pkg/report/report_test.go b/pkg/report/report_test.go
index 4d3137175..983da5d1e 100644
--- a/pkg/report/report_test.go
+++ b/pkg/report/report_test.go
@@ -372,6 +372,6 @@ func TestFuzz(t *testing.T) {
 		"cleaned vnod\re",
 		"kernel\r:",
 	} {
-		Fuzz([]byte(data))
+		Fuzz([]byte(data)[:len(data):len(data)])
 	}
 }
diff --git a/prog/parse.go b/prog/parse.go
index 8a8fdf9b1..7a46322df 100644
--- a/prog/parse.go
+++ b/prog/parse.go
@@ -26,7 +26,7 @@ func (target *Target) ParseLog(data []byte) []*LogEntry {
 	for pos := 0; pos < len(data); {
 		nl := bytes.IndexByte(data[pos:], '\n')
 		if nl == -1 {
-			nl = len(data)
+			nl = len(data) - 1
 		} else {
 			nl += pos
 		}
diff --git a/prog/test/fuzz_test.go b/prog/test/fuzz_test.go
index b3a123749..3774b2fba 100644
--- a/prog/test/fuzz_test.go
+++ b/prog/test/fuzz_test.go
@@ -22,9 +22,11 @@ mutate4()
 mutate7()
 mutate8()
 `,
+		`E`,
 	} {
 		t.Logf("test #%v: %q", i, data)
-		FuzzDeserialize([]byte(data))
-		FuzzParseLog([]byte(data))
+		inp := []byte(data)[:len(data):len(data)]
+		FuzzDeserialize(inp)
+		FuzzParseLog(inp)
 	}
 }