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

Dynamic engine code still active #14

Closed
lumag opened this issue Jul 16, 2014 · 4 comments
Closed

Dynamic engine code still active #14

lumag opened this issue Jul 16, 2014 · 4 comments

Comments

@lumag
Copy link
Contributor

lumag commented Jul 16, 2014

Even with OPENSSL_NO_DYNENGINE it is still possible to trigger dynamic engine loading code:

../apps/openssl engine -engine padlock
3073939132:error:260B6084:engine routines:DYNAMIC_LOAD:dso not found:../../portable/crypto/engine/eng_dyn.c:454:
3073939132:error:2606A074:engine routines:ENGINE_by_id:no such engine:../../portable/crypto/engine/eng_list.c:414:id=-engine
3073939132:error:260B6084:engine routines:DYNAMIC_LOAD:dso not found:../../portable/crypto/engine/eng_dyn.c:454:
3073939132:error:2606A074:engine routines:ENGINE_by_id:no such engine:../../portable/crypto/engine/eng_list.c:414:id=padlock
@busterb
Copy link
Contributor

busterb commented Oct 29, 2014

I experimented with replacing the dynamic engine code with no-op stubs. Will work on it a bit and see what the rest of the team thinks.

@busterb
Copy link
Contributor

busterb commented Dec 8, 2014

Disabling this caused some NULL pointer warnings in the engine iteration code. A corner case? Still looking at it.

@busterb
Copy link
Contributor

busterb commented Mar 4, 2015

This will likely go away in 2.2.x, once the cvs tree unlocks again for the next openbsd release.

@busterb
Copy link
Contributor

busterb commented Jun 19, 2015

The dynamic engine loader code is now removed, thanks!

@busterb busterb closed this as completed Jun 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants