Fix the bug of out-of-bound write (#10257)

* Fix the bug of out-of-bound write
* remove cgo dependencies, cgo and recover do not work together, I will subsequently remove all cgo's from make

Approved by: @aunjgr, @fengttt

pkg/common/hashmap/inthashmap.go

@@ -122,11 +122,19 @@ func fillKeys[T types.FixedSizeT](m *IntHashMap, vec *vector.Vector, size uint32
 		}
 	} else if vec.IsConst() {
 		ptr := (*T)(vector.GetPtrAt(vec, 0))
-		for i := 0; i < n; i++ {
-			*(*int8)(unsafe.Add(unsafe.Pointer(&keys[i]), keyOffs[i])) = 0
-			*(*T)(unsafe.Add(unsafe.Pointer(&keys[i]), keyOffs[i]+1)) = *ptr
+		// The old code was too stupid and would lead to out-of-bounds writing
+		if m.hasNull {
+			for i := 0; i < n; i++ {
+				*(*T)(unsafe.Add(unsafe.Pointer(&keys[i]), keyOffs[i])) = *ptr
+			}
+			uint32AddScalar(1+size, keyOffs[:n], keyOffs[:n])
+		} else {
+			for i := 0; i < n; i++ {
+				*(*int8)(unsafe.Add(unsafe.Pointer(&keys[i]), keyOffs[i])) = 0
+				*(*T)(unsafe.Add(unsafe.Pointer(&keys[i]), keyOffs[i]+1)) = *ptr
+			}
+			uint32AddScalar(size, keyOffs[:n], keyOffs[:n])
 		}
-		uint32AddScalar(size, keyOffs[:n], keyOffs[:n])
 	} else if !vec.GetNulls().Any() {
 		if m.hasNull {
 			for i := 0; i < n; i++ {

pkg/vectorize/sum/sum.go

@@ -14,13 +14,6 @@
 
 package sum
 
-/*
-#include "mo.h"
-#cgo CFLAGS: -I../../../cgo
-#cgo LDFLAGS: -L../../../cgo -lmo -lm
-*/
-import "C"
-
 import (
 	"github.com/matrixorigin/matrixone/pkg/container/nulls"
 	"github.com/matrixorigin/matrixone/pkg/container/types"