-
Notifications
You must be signed in to change notification settings - Fork 347
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
Prevent accessing of global c++ objects once they are deleted #742
base: develop
Are you sure you want to change the base?
Conversation
Softhsm2 seems to be rarely maintained these days and has a bug that leads to segfaults when running through Openssl library destructors. Add the Patch from [1] until this is fixed upstream. [1] softhsm/SoftHSMv2#742 Signed-off-by: Johannes Wiesboeck <johannes.wiesboeck@aisec.fraunhofer.de>
Softhsm2 seems to be rarely maintained these days and has a bug that leads to segfaults when running through Openssl library destructors. Add the Patch from [1] until this is fixed upstream. [1] softhsm/SoftHSMv2#742 Signed-off-by: Johannes Wiesboeck <johannes.wiesboeck@aisec.fraunhofer.de>
Softhsm2 seems to be rarely maintained these days and has a bug that leads to segfaults when running through Openssl library destructors. Add the Patch from [1] until this is fixed upstream. [1] softhsm/SoftHSMv2#742 Signed-off-by: Johannes Wiesboeck <johannes.wiesboeck@aisec.fraunhofer.de>
Softhsm2 seems to be rarely maintained these days and has a bug that leads to segfaults when running through Openssl library destructors. Set the SRC_URI to the current development version as the last "release" is from 2020. Add the Patch from [1] until this is fixed upstream. [1] softhsm/SoftHSMv2#742 Signed-off-by: Johannes Wiesboeck <johannes.wiesboeck@aisec.fraunhofer.de>
hmm reading through #729 it seems to be a fix for openSSL's behavior where its using [1] https://en.cppreference.com/w/cpp/utility/program/atexit |
Please rebase on develop and mark as ready when ready. |
Fixes (Maybe) softhsm#729. Reset objects_deleted after reset is called.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if there is a better way but I guess this work unless we look into some refactoring.
Maybe add a comment with the reason for using a objects_deleted
.
One flag isInitialised, another flags objects_deleted , tomorrow yet another flag... :( . |
Almost all issues shows call-stack that starts "at exit", call softhsm ...CloseSession... and then crash i.e., bus error, or segmentation fault, or core dump. More or less this is expected when a project uses unreliable and non-portable functionality like at-exit. In addition refuses to proposes portable solution. For instance cygwin hang is because of "at exit" functionality. With similar defects third parties try to force project to fix external issues. May be just to stop processing when is detected "lazy" clean-up request. I mean to check for null pointers in close session. Off topic: Long time ago in pkix-ssh implementation allows use of identity (keys) from engine. This is an openssl loadable module and one thing is clear indirectly - in general module cannot be considered as stateless. It may open database and so close of database should be triggered as well. It may use network connections, devices , and etc.. Clean shutdown of used resources is preferred. |
Fixes (Maybe) #729.
Reset objects_deleted after reset is called.