Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Secure wipe context manager #843

Closed
wants to merge 2 commits into from
Closed

Secure wipe context manager #843

wants to merge 2 commits into from

Conversation

public
Copy link
Member

@public public commented Mar 22, 2014

This can only cope with bytes but might still be useful. No docs we might want the context manager to return a cdata or something too?

See also #7

@jenkins-cryptography
Copy link

Test FAILed.
Refer to this link for build results: https://jenkins.cryptography.io/job/cryptography-pr-experimental/1241/

@public
Copy link
Member Author

public commented Mar 22, 2014

As pointed out by @alex (and all the failing tests) this also doesn't work on PyPy.

@alex
Copy link
Member

alex commented Mar 22, 2014

And even when it appears to work on CPython, it's super undefined behavior.

On Sat, Mar 22, 2014 at 10:14 AM, Alex Stapleton
notifications@github.comwrote:

As pointed out by @alex https://github.com/alex (and all the failing
tests) this also doesn't work on PyPy.


Reply to this email directly or view it on GitHubhttps://github.com//pull/843#issuecomment-38357701
.

"I disapprove of what you say, but I will defend to the death your right to
say it." -- Evelyn Beatrice Hall (summarizing Voltaire)
"The people's good is the highest law." -- Cicero
GPG Key fingerprint: 125F 5C67 DFE9 4084

@jenkins-cryptography
Copy link

Test FAILed.
Refer to this link for build results: https://jenkins.cryptography.io/job/cryptography-pr-experimental/1244/

@public
Copy link
Member Author

public commented Mar 22, 2014

Maybe we should just document that Python sucks at memory wiping somewhere then?

@public
Copy link
Member Author

public commented Mar 22, 2014

I'm going to close this in favour of #845 :(

@public public closed this Mar 22, 2014
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants