fix(runtime): roll back snapshot view binds on failure

Signed-off-by: sidneychang <2190206983@qq.com>

Author: sidneychang

pkg/unikontainers/block.go

@@ -260,14 +260,24 @@ func (b blockRootfs) preSetup() error {
 
 	bundleDir := filepath.Dir(b.monRootfs)
 	var viewTargets []string
+	viewMountsCommitted := false
+	defer func() {
+		if viewMountsCommitted || len(viewTargets) == 0 {
+			return
+		}
+		rollbackPerContainerViewTargets(viewTargets)
+		if err := removePerContainerViewMountsState(bundleDir); err != nil {
+			uniklog.WithError(err).Warn("failed to remove snapshot view mount state during rollback")
+		}
+	}()
+
 	fromPerContainerView, err := tryRunOnPerContainerView(bundleDir, func(root string) error {
 		var bindErr error
 		viewTargets, bindErr = bindBootArtifactsToMonRootfs(root, b.monRootfs, b.kernelPath, b.initrdPath, b.uruncJSONPath)
 		return bindErr
 	})
 	if err != nil {
 		if len(viewTargets) > 0 {
-			rollbackPerContainerViewTargets(viewTargets)
 			return fmt.Errorf("snapshot view boot artifact bind completed but cleanup failed: %w", err)
 		}
 		uniklog.WithError(err).Warn("snapshot view unavailable; falling back to legacy boot file extraction")
@@ -286,6 +296,7 @@ func (b blockRootfs) preSetup() error {
 		if err := mount.Unmount(b.mountedPath); err != nil {
 			return fmt.Errorf("failed to unmount rootfs: %w", err)
 		}
+		viewMountsCommitted = true
 	} else {
 		if err := removePerContainerViewMountsState(bundleDir); err != nil {
 			return err