Access violation if GCM mode is set after cipher is initialized
@MHumm 已經在處理了。
開始於 2024年5月8日。
評估
這個 Issue 還沒有評估資料。
描述
To Reproduce
- Open demo project Cipher_Console
- Change Cipher.Mode to cmGCM
- Run the code. You receive EAccessViolation
Usually, if you change the cipher mode after the cipher has been initialized, then TDECCipher.SetMode should call Done on the cipher and re-initialize it.
GCM Encode requires that TGCM.Init is called and has set FEncryptionMethod := EncryptionMethod. However, if Ciper.Mode is set after the cipher has been initialized, then TGCM is created but TGCM.Init has not been called.
procedure TDECCipherModes.InitMode;
begin
if FMode = TCipherMode.cmGCM then
begin
if Context.BlockSize = 16 then
FGCM := TGCM.Create
else
// GCM requires a cipher with 128 bit block size
raise EDECCipherException.CreateResFmt(@sInvalidBlockSize,
[128, GetEnumName(TypeInfo(TCipherMode),
Integer(FMode))]);
end
else
if Assigned(FGCM) then
FreeAndNil(FGCM);
end;
The program flow if Mode has been set before init:
- TDECCipher.Init()
- TDECCipher.Init()
- TDECCipherModes.OnAfterInitVectorInitialized ( calls
FGCM.Init(self.DoEncode, OriginalInitVector)) - TGCM.Init
The program flow if Mode has been set after init:
- TDECCipher.SetMode
- TDECCipherModes.InitMode
- TGCM.Create, but not TGCM.Init ( I am not sure where I get
OriginalInitVectorfrom )
So... what do we do? We probably need to preserve the InitVector, so we can call TGCM.Init with the correct parameters.
The other solution is to forbid to change the mode after the cipher has been initialized. But this is a code breaking change.
- 主要語言
- HTML
- 星號
- 286
- 分支
- 78
- 平均合併
- 1 天 4 小時
- 30 天內合併 PR
- 4
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
MHumm/DelphiEncryptionCompendium 的其他 Issue
-
難度 2/5 1-3 小時 新手友好度 68/100
-
CRC64 - PKI Addition可能重新可做 @MHumm 於 149 天前認領，目前沒有進行中的 PR。 未關閉enhancement help wanted
MHumm/DelphiEncryptionCompendium#93 · 1 則留言 · 已指派 1 人 ·
-
Missing PUInt8Array type in the GetIt version.可能重新可做 @MHumm 於 160 天前認領，目前沒有進行中的 PR。 未關閉bug delphi
MHumm/DelphiEncryptionCompendium#92 · 2 則留言 · 已指派 1 人 ·
-
Help with demo未關閉question
難度 4/5 1-2 天 新手友好度 25/100
MHumm/DelphiEncryptionCompendium#91 · 6 則留言 ·
-
Chacha20 Poly1305可能重新可做 @MHumm 於 459 天前認領，目前沒有進行中的 PR。 未關閉enhancement
MHumm/DelphiEncryptionCompendium#89 · 1 則留言 · 已指派 1 人 ·