Fix resource leaks in node_crypto.cc #1097

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@bnoordhuis
Member

Example:

c = require('crypto').createCipher('des-ecb', '12345678');
c.update('foo');
// don't call c.final()

Crypto objects fail to free up resources (potentially leaving sensitive information in memory? openssl man pages warn about it) if the final() method isn't called.

@ry ry added a commit that closed this pull request May 25, 2011
@bnoordhuis @ry bnoordhuis + ry Fix resource leaks in node_crypto.cc
Fixes #1097.
eb4c9ed
@ry ry closed this in eb4c9ed May 25, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment