fix(hooks): prevent panic from loop var capture

Fix panic from accessing hooks[i] in async execution after loop.

Use captured parameter h instead.

Fixes: #418

Co-authored-by: hemang1404 <hemangsharrma@gmail.com>
Signed-off-by: ishaanxgupta <ishaankone@gmail.com>

Author: ishaanxgupta

.github/contributors.yaml

@@ -53,6 +53,9 @@ users:
   ishaanxgupta:
     name: Ishaan Gupta
     email: ishaankone@gmail.com
+  hemang1404:
+    name: Hemang Sharrma
+    email: hemangsharrma@gmail.com
   ilp-sys:
     name: Jiwoo Ahn
     email: ikwydls1314@gmail.com

pkg/unikontainers/unikontainers.go

@@ -777,8 +777,8 @@ func (u *Unikontainer) executeHooksConcurrently(name string, hooks []specs.Hook,
 				uniklog.WithFields(logrus.Fields{
 					"id":    u.State.ID,
 					"name":  name,
-					"path":  hooks[i].Path,
-					"args":  hooks[i].Args,
+					"path":  h.Path,
+					"args":  h.Args,
 					"error": err,
 				}).Error("Executing hook failed")
 				errChan <- err