Prevent nil panic on exec restore hooks (vmware-tanzu#5675)

* Prevent nil panic on exec restore hooks Signed-off-by: Dylan Murray <dymurray@redhat.com>
dymurray · Dec 13, 2022 · 55873c1 · 55873c1
1 parent ffc9845
commit 55873c1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/changelogs/unreleased/5675-dymurray b/changelogs/unreleased/5675-dymurray
@@ -0,0 +1 @@
+Prevent nil panic on exec restore hooks
diff --git a/pkg/controller/restore_controller.go b/pkg/controller/restore_controller.go
@@ -342,10 +342,12 @@ func (c *restoreController) validateAndComplete(restore *api.Restore, pluginMana
 	}
 	for _, resource := range restoreHooks {
 		for _, h := range resource.RestoreHooks {
-			for _, container := range h.Init.InitContainers {
-				err = hook.ValidateContainer(container.Raw)
-				if err != nil {
-					restore.Status.ValidationErrors = append(restore.Status.ValidationErrors, err.Error())
+			if h.Init != nil {
+				for _, container := range h.Init.InitContainers {
+					err = hook.ValidateContainer(container.Raw)
+					if err != nil {
+						restore.Status.ValidationErrors = append(restore.Status.ValidationErrors, err.Error())
+					}
 				}
 			}
 		}