linters: add makezero and fix a related bug

Makezero is a linter to ensure that slices are not initialized with
non-zero length. Since in Go, we most of the time use append on the
slice, initializing it with non-zero length can tend to create a slice
full of zero that will clobber the beginning of the slice.

In the bug fixed here, we created a list with 300+ zeros and then append
after them, only to skip the zero in the calling method. Usually we want
to prealloc the slice with a capacity of len but a length of 0, i.e.
replace make([]int, len(nums)) with make([]int, 0, len(nums)).  See more
info here https://github.com/ashanbrown/makezero.

Signed-off-by: Mahe Tardy <[email protected]>

.golangci.yml

@@ -19,6 +19,7 @@ linters:
     - staticcheck
     - unused
     - gosimple
+    - makezero
 
 linters-settings:
   goheader:

pkg/btf/validation.go

@@ -325,10 +325,6 @@ func GetSyscallsList() ([]string, error) {
 	for _, key := range syscallinfo.SyscallsNames() {
 		var fn *btf.Func
 
-		if key == "" {
-			continue
-		}
-
 		sym, err := arch.AddSyscallPrefix(key)
 		if err != nil {
 			return []string{}, err

pkg/syscallinfo/syscallinfo.go

@@ -44,7 +44,7 @@ var syscallIDs = func() map[string]int {
 }()
 
 func SyscallsNames() []string {
-	ret := make([]string, len(syscallNames))
+	ret := make([]string, 0, len(syscallNames))
 
 	for _, v := range syscallNames {
 		ret = append(ret, v)