From e961d16c3e35fd78acdfaf5531fc1c1365c24a97 Mon Sep 17 00:00:00 2001
From: Aleksandr Nogikh <nogikh@google.com>
Date: Thu, 30 Jan 2025 11:49:34 +0100
Subject: all: clarify the error in case of ExecFailure

Whenever the status is set, also include the reason. It should help
easier debug execution and machine check time problems.
---
 syz-manager/snapshot.go | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

(limited to 'syz-manager')

diff --git a/syz-manager/snapshot.go b/syz-manager/snapshot.go
index a7e72132f..21e6ed62b 100644
--- a/syz-manager/snapshot.go
+++ b/syz-manager/snapshot.go
@@ -104,7 +104,10 @@ func (mgr *Manager) snapshotRun(inst *vm.Instance, builder *flatbuffers.Builder,
 	progData, err := req.Prog.SerializeForExec()
 	if err != nil {
 		queue.StatExecBufferTooSmall.Add(1)
-		return &queue.Result{Status: queue.ExecFailure}, nil, nil
+		return &queue.Result{
+			Status: queue.ExecFailure,
+			Err:    fmt.Errorf("program serialization failed: %w", err),
+		}, nil, nil
 	}
 	msg := flatrpc.SnapshotRequestT{
 		ExecFlags: req.ExecOpts.ExecFlags,
-- 
cgit mrf-deployment