FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

crypto: allow GCM ciphers to have longer IV length by bnoordhuis · Pull Request #7705 · nodejs/node · GitHub

/ node Public

crypto: allow GCM ciphers to have longer IV length - #7705

Closed
bnoordhuis wants to merge 1 commit into
nodejs:v4.x-stagingfrom
bnoordhuis:backport-pr6376-v4.x
Closed

crypto: allow GCM ciphers to have longer IV length#7705
bnoordhuis wants to merge 1 commit into
nodejs:v4.x-stagingfrom
bnoordhuis:backport-pr6376-v4.x

Conversation

bnoordhuis commented Jul 13, 2016
edited
Loading

Copy link
Copy Markdown
Member

GCM cipher IV length can be >=1 bytes.
When not the default 12 bytes (96 bits) sets the IV length using
`EVP_CIPHER_CTX_ctrl` with type `EVP_CTRL_GCM_SET_IVLEN`

PR-URL: nodejs#6376
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Shigeki Ohtsu <ohtsu@iij.ad.jp>
bnoordhuis added crypto Issues and PRs related to the crypto subsystem. v4.x labels Jul 13, 2016
nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Jul 13, 2016

Copy link
Copy Markdown
Contributor

landed in 8c4f6c1

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL