Commit e8c66f9
crypto: remove unneeded guard
createCFRGRaw() will always return an object. The error for an undefined
return value will never occur.
PR-URL: #43856
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Filip Skokan <panva.ip@gmail.com>
Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent c92135a commit e8c66f9
2 files changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -291,8 +291,6 @@ async function cfrgImportKey( | |||
| 291 | 291 | case 'raw': { | |
| 292 | 292 | verifyAcceptableCfrgKeyUse(name, 'public', usagesSet); | |
| 293 | 293 | keyObject = createCFRGRawKey(name, keyData, true); | |
| 294 | - if (keyObject === undefined) | ||
| 295 | - throw lazyDOMException('Unable to import CFRG key', 'OperationError'); | ||
| 296 | 294 | break; | |
| 297 | 295 | } | |
| 298 | 296 | } | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -85,7 +85,7 @@ for (const m of [[kKeyEncodingPKCS1, 'pkcs1'], [kKeyEncodingPKCS8, 'pkcs8'], | |||
| 85 | 85 | encodingNames[m[0]] = m[1]; | |
| 86 | 86 | ||
| 87 | 87 | // Creating the KeyObject class is a little complicated due to inheritance | |
| 88 | - // and that fact that KeyObjects should be transferrable between threads, | ||
| 88 | + // and the fact that KeyObjects should be transferrable between threads, | ||
| 89 | 89 | // which requires the KeyObject base class to be implemented in C++. | |
| 90 | 90 | // The creation requires a callback to make sure that the NativeKeyObject | |
| 91 | 91 | // base class cannot exist without the other KeyObject implementations. | |
0 commit comments