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

Use Improved Key Reuse Procedure #15

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Use Improved Key Reuse Procedure #15

wants to merge 6 commits into from

Conversation

dndx
Copy link
Owner

@dndx dndx commented Jan 24, 2013

No description provided.

@dndx
Copy link
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

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.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dndx @cyfdecyf