fix panic found by fuzzer (#8)

fuzzer found panic for trivial empty table `|||` - let's fix it
sivukhin · Sep 5, 2024 · 9170633 · 9170633
2 parents c976456 + d392f90
commit 9170633
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/djot_parser/djot_ast.go b/djot_parser/djot_ast.go
@@ -468,7 +468,7 @@ func buildDjotAst(
 				for s := i + 1; s < i+openToken.JumpToPair; s++ {
 					if list[s].Type == djot_tokenizer.PipeTableSeparator {
 						columns++
-						content := document[list[s].Start+1 : list[s+list[s].JumpToPair].End-1]
+						content := document[list[s].Start+1 : list[s+list[s].JumpToPair].Start]
 						dashCount := bytes.Count(content, []byte("-"))
 						if dashCount == 0 {
 							continue

diff --git a/djot_parser/testdata/fuzz/FuzzDjotE2E/e6a6fccfa2e3278f b/djot_parser/testdata/fuzz/FuzzDjotE2E/e6a6fccfa2e3278f
@@ -0,0 +1,2 @@
+go test fuzz v1
+string("|||")