diff options
| author | Dmitry Vyukov <dvyukov@google.com> | 2026-03-11 05:56:50 +0100 |
|---|---|---|
| committer | Aleksandr Nogikh <nogikh@google.com> | 2026-03-11 10:29:00 +0000 |
| commit | e09601ff0fad36b90ec8539f5e3d0fcfc615d42d (patch) | |
| tree | 8d071c8f641281cdde1eb39c482cbe869ff1eac9 /pkg/aflow/action/kernel | |
| parent | cf229fb049a95c12f36c391dad739f763f876219 (diff) | |
syz-agent: systematically show verbose error messages
Currently we added custom code to kernel build action,
and few others to expose verbose errors from executed binaries (notably make).
But lots of other binary executions missing this logic,
e.g. for git failure we currently see unuseful:
failed to run ["git" "fetch" "--force" "--tags" exit status 128
Instead of adding more and more custom code to do the same,
remove the custom code and always add verbose output
in syz-agent and tools/syz-aflow.
Diffstat (limited to 'pkg/aflow/action/kernel')
| -rw-r--r-- | pkg/aflow/action/kernel/build.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkg/aflow/action/kernel/build.go b/pkg/aflow/action/kernel/build.go index c10dc7d93..4d37dca86 100644 --- a/pkg/aflow/action/kernel/build.go +++ b/pkg/aflow/action/kernel/build.go @@ -52,8 +52,8 @@ func BuildKernel(buildDir, srcDir, cfg string, cleanup bool) error { "ccache", buildDir, runtime.NumCPU()) const compileCommands = "compile_commands.json" makeArgs = append(makeArgs, "-s", path.Base(image), compileCommands) - if out, err := osutil.RunCmd(time.Hour, srcDir, "make", makeArgs...); err != nil { - return aflow.FlowError(fmt.Errorf("make failed: %w\n%s", err, out)) + if _, err := osutil.RunCmd(time.Hour, srcDir, "make", makeArgs...); err != nil { + return aflow.FlowError(err) } if !cleanup { return nil |