gzhttp: Clamp compression level (#890)

Clamp compression levels to valid values.
klauspost · Nov 28, 2023 · c4e922d · c4e922d
1 parent bb3065b
commit c4e922d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/gzhttp/writer/gzkp/gzkp.go b/gzhttp/writer/gzkp/gzkp.go
@@ -24,6 +24,12 @@ func init() {
 // poolIndex maps a compression level to its index into gzipWriterPools. It
 // assumes that level is a valid gzip compression level.
 func poolIndex(level int) int {
+	if level > gzip.BestCompression {
+		level = gzip.BestCompression
+	}
+	if level < gzip.StatelessCompression {
+		level = gzip.BestSpeed
+	}
 	return level - gzip.StatelessCompression
 }
 

diff --git a/gzhttp/writer/gzstd/stdlib.go b/gzhttp/writer/gzstd/stdlib.go
@@ -24,6 +24,12 @@ func init() {
 // poolIndex maps a compression level to its index into gzipWriterPools. It
 // assumes that level is a valid gzip compression level.
 func poolIndex(level int) int {
+	if level > gzip.BestCompression {
+		level = gzip.BestCompression
+	}
+	if level < gzip.HuffmanOnly {
+		level = gzip.BestSpeed
+	}
 	return level - gzip.HuffmanOnly
 }