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

Fixed invalid type of argument for HSM_PKCS11_set_attr_int() #7

Merged
merged 9 commits into from
Sep 19, 2016

Conversation

a157634
Copy link
Contributor

@a157634 a157634 commented Jan 31, 2015

Changed from "int" to "CK_ULONG".
This fixes problems on 64bit targets due to different size of int (4 bytes) to unsigned long (8 bytes).

a157634 and others added 9 commits January 26, 2015 18:28
… to <CK_ULONG>.

This fixes problems on 64bit targets due to different size of int (4 bytes) to unsigned long (8 bytes).
Introduced r/o memory BIO for __get_data_callback().
The privously used r/w memory BIO is very slow in loading large CRLs.
The usage of a r/o memory BIO speeds up the process for loading such large files.
See also https://www.openssl.org/docs/crypto/BIO_s_mem.html#notes

Furthermore fixed:

- Fix memory leak in PKI_X509_STACK_get_mem()

- Code cleanup of unused variable in PKI_X509_put_mem_value()
Fixed typo in example config files
… to <CK_ULONG>.

This fixes problems on 64bit targets due to different size of int (4 bytes) to unsigned long (8 bytes).
Introduced r/o memory BIO for __get_data_callback().
The privously used r/w memory BIO is very slow in loading large CRLs.
The usage of a r/o memory BIO speeds up the process for loading such large files.
See also https://www.openssl.org/docs/crypto/BIO_s_mem.html#notes

Furthermore fixed:

- Fix memory leak in PKI_X509_STACK_get_mem()

- Code cleanup of unused variable in PKI_X509_put_mem_value()
Will be sperated in a dedicated pull request
@opencrypto opencrypto merged commit 90051ec into openca:master Sep 19, 2016
@a157634 a157634 deleted the libpki-b1 branch September 20, 2016 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants