Skip to content
This repository was archived by the owner on Apr 24, 2022. It is now read-only.

Use Improved Key Reuse Procedure#15

Open
dndx wants to merge 6 commits intodndx:masterfrom
cyfdecyf:master
Open

Use Improved Key Reuse Procedure#15
dndx wants to merge 6 commits intodndx:masterfrom
cyfdecyf:master

Conversation

@dndx
Copy link
Copy Markdown
Owner

@dndx dndx commented Jan 24, 2013

No description provided.

@dndx
Copy link
Copy Markdown
Owner Author

dndx commented Jan 24, 2013

Little problem here, in make_encryptor(), if tpl is NULL, the function will treat enc as tpl and initialize the encryption context. But in L203, !tpl->rc4_en may not be safe cause tpl will possibility be NULL. Probably will cause problem.

@cyfdecyf
Copy link
Copy Markdown

Well, I think L179 ensures tpl would not be NULL in L203. Otherwise L195 would have the same problem. Is there anything I missed?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants