Fix NodeJS and Python bindings on 32-bit system, off-by-one, fixes #707

tdewolff · May 22, 2024 · 4af426d · 4af426d
1 parent 530e414
commit 4af426d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/bindings/js/minify.go b/bindings/js/minify.go
@@ -34,7 +34,7 @@ func goStringArray(carr **C.char, length C.longlong) []string {
 	}
 
 	strs := make([]string, length)
-	arr := (*[1 << 31]*C.char)(unsafe.Pointer(carr))[:length:length]
+	arr := (*[1<<31 - 1]*C.char)(unsafe.Pointer(carr))[:length:length]
 	for i := 0; i < int(length); i++ {
 		strs[i] = C.GoString(arr[i])
 	}

diff --git a/bindings/py/minify.go b/bindings/py/minify.go
@@ -34,7 +34,7 @@ func goStringArray(carr **C.char, length C.longlong) []string {
 	}
 
 	strs := make([]string, length)
-	arr := (*[1 << 31]*C.char)(unsafe.Pointer(carr))[:length:length]
+	arr := (*[1<<31 - 1]*C.char)(unsafe.Pointer(carr))[:length:length]
 	for i := 0; i < int(length); i++ {
 		strs[i] = C.GoString(arr[i])
 	}