Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syz-ci: job: reproducer consistency is invalid errors #5590

Open
a-nogikh opened this issue Dec 11, 2024 · 3 comments
Open

syz-ci: job: reproducer consistency is invalid errors #5590

a-nogikh opened this issue Dec 11, 2024 · 3 comments
Labels

Comments

@a-nogikh
Copy link
Collaborator

We're seeing job: reproducer consistency is invalid errors once in a while.

syzkaller/syz-ci/jobs.go

Lines 416 to 417 in ff949d2

{"reproducer consistency", (len(req.ReproOpts) != 0 && len(req.ReproSyz) != 0) ||
(len(req.ReproOpts) == 0 && len(req.ReproSyz) == 0)},

Last time it happened on this bug: https://syzkaller.appspot.com/bug?extid=27209997e4015fb4702e

There are indeed a number of crash reports with C reports, but empty syz reports. I don't immediately understand how it could have happened.

@a-nogikh a-nogikh added the bug label Dec 11, 2024
@dvyukov
Copy link
Collaborator

dvyukov commented Dec 11, 2024

I guess we removed all syscalls during minimization, and it still crashed :)

void execute_one(void)
{
  if (write(1, "executing program\n", sizeof("executing program\n") - 1)) {
  }
}

@a-nogikh
Copy link
Collaborator Author

Hmm, then this logic apparently does not work:

res.Prog, _ = prog.Minimize(res.Prog, -1, mode, func(p1 *prog.Prog, callIndex int) bool {
if len(p1.Calls) == 0 {
// We do want to keep at least one call, otherwise tools/syz-execprog
// will immediately exit.
return false
}

In any case, we now already have such crashes in our database. We either delete them and patch the code never to report them or we get rid of the reproducer consistency check completely.

@dvyukov
Copy link
Collaborator

dvyukov commented Dec 11, 2024

We have another prog.Minimize call there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants