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

crypto: fix rsa-pss one-shot sign/verify error handling #39830

Closed
wants to merge 1 commit into from

Conversation

panva
Copy link
Member

@panva panva commented Aug 21, 2021

fixes #39822

cc @nodejs/crypto

@panva panva added crypto Issues and PRs related to the crypto subsystem. webcrypto labels Aug 21, 2021
@panva panva requested review from jasnell and tniessen August 21, 2021 11:28
@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels Aug 21, 2021
@panva panva added request-ci Add this label to start a Jenkins CI on a PR. and removed needs-ci PRs that need a full CI run. labels Aug 21, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 21, 2021
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Aug 21, 2021

@panva panva added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Aug 21, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 28, 2021
@github-actions
Copy link
Contributor

Commit Queue failed
- Loading data for nodejs/node/pull/39830
✔  Done loading data for nodejs/node/pull/39830
----------------------------------- PR info ------------------------------------
Title      crypto: fix rsa-pss one-shot sign/verify error handling (#39830)
Author     Filip Skokan  (@panva)
Branch     panva:fix-oneshot-pss -> nodejs:master
Labels     crypto, c++, author ready, webcrypto
Commits    1
 - crypto: fix rsa-pss one-shot sign/verify error handling
Committers 1
 - Filip Skokan 
PR-URL: https://github.com/nodejs/node/pull/39830
Reviewed-By: James M Snell 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/39830
Reviewed-By: James M Snell 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Sat, 21 Aug 2021 11:28:00 GMT
   ✔  Approvals: 1
   ✔  - James M Snell (@jasnell) (TSC): https://github.com/nodejs/node/pull/39830#pullrequestreview-735495160
   ✖  This PR needs to wait 3 more hours to land (or 0 hours if there is one more approval)
   ✔  Last GitHub Actions successful
   ℹ  Last Full PR CI on 2021-08-21T13:18:37Z: https://ci.nodejs.org/job/node-test-pull-request/39618/
- Querying data for job/node-test-pull-request/39618/
✔  Build data downloaded
   ✔  Last Jenkins CI successful
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/1176709244

@github-actions github-actions bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Aug 28, 2021
@panva panva added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Aug 28, 2021
@github-actions github-actions bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 28, 2021
@github-actions
Copy link
Contributor

Landed in e5670f4...b5816cd

@github-actions github-actions bot closed this Aug 28, 2021
nodejs-github-bot pushed a commit that referenced this pull request Aug 28, 2021
fixes #39822

PR-URL: #39830
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Sep 6, 2021
fixes #39822

PR-URL: #39830
Reviewed-By: James M Snell <jasnell@gmail.com>
targos pushed a commit that referenced this pull request Sep 6, 2021
fixes #39822

PR-URL: #39830
Reviewed-By: James M Snell <jasnell@gmail.com>
@targos targos mentioned this pull request Sep 6, 2021
@panva panva deleted the fix-oneshot-pss branch October 13, 2022 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. webcrypto
Projects
None yet
Development

Successfully merging this pull request may close these issues.

segmentation fault with rsa-pss when pss params don't match the operation
3 participants