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

OpenSSH: signature verification nondeterministic spurious failure #59242

Closed
bwendling opened this issue Nov 29, 2022 · 16 comments · Fixed by llvm/llvm-project-release-prs#223
Closed
Assignees
Labels
bug Indicates an unexpected problem or unintended behavior llvm:codegen miscompilation release:backport release:merged

Comments

@bwendling
Copy link
Collaborator

There appears to be a bug with -fzero-call-used-regs in OpenSSH

https://bugzilla.mindrot.org/show_bug.cgi?id=3475

@bwendling bwendling self-assigned this Nov 29, 2022
@bwendling bwendling added bug Indicates an unexpected problem or unintended behavior and removed new issue labels Nov 29, 2022
@llvmbot
Copy link
Collaborator

llvmbot commented Nov 29, 2022

@llvm/issue-subscribers-bug

@bwendling
Copy link
Collaborator Author

/s : @nickdesaulniers @kees

@thesamesam
Copy link
Member

See also #57692.

@thesamesam
Copy link
Member

cc @nikic

@nickdesaulniers
Copy link
Member

#57692 has been closed. Is this fixed, or is there more than one bug with -fzero-call-used-regs?

@nikic
Copy link
Contributor

nikic commented Dec 8, 2022

The bugzilla comments indicate that this still occurs with 15.0.6 which should include the aforementioned patch, so this should be another issue.

@nickdesaulniers
Copy link
Member

nickdesaulniers commented Dec 8, 2022

ClangBuiltLinux/linux#1766 is another crash that seems to be with clang 15.0.6. I've asked the reporter for more info though to confirm.

@bwendling
Copy link
Collaborator Author

It looks like https://reviews.llvm.org/D139679 fixes the OpenSSH issue.

@nickdesaulniers
Copy link
Member

14d4cdd has landed. I would kindly ask the release folks if this warrants a 15.0.7 release; it's a pretty bad bug affecting both the Linux kernel and OpenSSL.

cc @tstellar

@nickdesaulniers
Copy link
Member

cc @tru

@tru
Copy link
Collaborator

tru commented Dec 22, 2022

Hm, we have generally closed the book on 15.x and are planning to rev up 16.x release in the beginning of next year. This does seem bad though, so I am sure we can consider it.

Tom is out until early January and I suspect most release testers are as well, so I doubt we can get a release out before then. I'll note it down and talk to Tom about it when he's back online.

@tstellar
Copy link
Collaborator

tstellar commented Jan 4, 2023

This seems severe enough to me to warrant another release. I think we could do one Tue Jan 10.

@thesamesam
Copy link
Member

There's also #59792 which is quite serious.

@tstellar
Copy link
Collaborator

tstellar commented Jan 6, 2023

/cherry-pick 14d4cdd

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 6, 2023

/branch llvm/llvm-project-release-prs/issue59242

@llvmbot
Copy link
Collaborator

llvmbot commented Jan 6, 2023

/pull-request llvm/llvm-project-release-prs#223

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior llvm:codegen miscompilation release:backport release:merged
Projects
Development

Successfully merging a pull request may close this issue.

8 participants