Merge pull request #102 from grafana/20220719_fix-bug-with-string-sli…

…ce-length Fix string slice length bug
grafana · Jul 19, 2022 · 0bdac21 · 0bdac21
2 parents de676bc + a54274b
commit 0bdac21
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/pkg/firedb/strings.go b/pkg/firedb/strings.go
@@ -15,7 +15,14 @@ func (*stringsHelper) key(s string) string {
 }
 
 func (*stringsHelper) addToRewriter(r *rewriter, m idConversionTable) {
-	r.strings = make(stringConversionTable, len(m))
+	var maxID int64
+	for id := range m {
+		if id > maxID {
+			maxID = id
+		}
+	}
+	r.strings = make(stringConversionTable, maxID+1)
+
 	for x, y := range m {
 		r.strings[x] = y
 	}