diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2025-01-30 11:49:34 +0100 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2025-01-30 11:03:14 +0000 |
| commit | e961d16c3e35fd78acdfaf5531fc1c1365c24a97 (patch) | |
| tree | 37f72c1514b9f3938983da79380a3b3f8bbaf485 /syz-manager | |
| parent | 9c8ab8458ec658aa3007c0e8c1695907c8d8c39f (diff) | |
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.
Diffstat (limited to 'syz-manager')
| -rw-r--r-- | syz-manager/snapshot.go | 5 |
1 files changed, 4 insertions, 1 deletions
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, |