-
Notifications
You must be signed in to change notification settings - Fork 264
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
LibreSSL 3.7.x doesn't have NID_sm2 #841
Comments
SM2 is a digital signature algorithm specified by China: https://datatracker.ietf.org/doc/html/draft-shen-sm2-ecdsa-02 LibreSSL has code to support this, however it needs further work before it could be enabled (and unfortunately that is currently low on the list of priorities). Instead of using |
The NID_sm2 algorithm is not currently supported by LibreSSL which causes build failures. However instead of checking the OpenSSL version number it is possible to just check if NID_sm2 is defined instead. This way it will be automatically enabled when LibreSSL does support it and disabled in the event the OpenSSL stops supporting it. LibreSSL issue: libressl/portable#841
Thanks for the information and suggestion. I made a new PR for tpm2-tss with just this change. tpm2-software/tpm2-tss#2602 I will leave this issue open for now as a way to track the SM2 support, but I am fine with it being a low priority. |
The NID_sm2 algorithm is not currently supported by LibreSSL which causes build failures. However instead of checking the OpenSSL version number it is possible to just check if NID_sm2 is defined instead. This way it will be automatically enabled when LibreSSL does support it and disabled in the event the OpenSSL stops supporting it. LibreSSL issue: libressl/portable#841 Signed-off-by: orbea <[email protected]>
Issue #635 already exists for this purpose. As such, I'll close this issue. |
The NID_sm2 algorithm is not currently supported by LibreSSL which causes build failures. However instead of checking the OpenSSL version number it is possible to just check if NID_sm2 is defined instead. This way it will be automatically enabled when LibreSSL does support it and disabled in the event the OpenSSL stops supporting it. LibreSSL issue: libressl/portable#841 Signed-off-by: orbea <[email protected]>
The NID_sm2 algorithm is not currently supported by LibreSSL which causes build failures. However instead of checking the OpenSSL version number it is possible to just check if NID_sm2 is defined instead. This way it will be automatically enabled when LibreSSL does support it and disabled in the event the OpenSSL stops supporting it. LibreSSL issue: libressl/portable#841 Signed-off-by: orbea <[email protected]>
The NID_sm2 algorithm is not currently supported by LibreSSL which causes build failures. However instead of checking the OpenSSL version number it is possible to just check if NID_sm2 is defined instead. This way it will be automatically enabled when LibreSSL does support it and disabled in the event the OpenSSL stops supporting it. LibreSSL issue: libressl/portable#841 Signed-off-by: orbea <[email protected]>
When building tpm2-tss it fails since
NID_sm2
is not implemented.https://github.com/tpm2-software/tpm2-tss/blob/3d3c9a81db1354fe75dd27f5a87551c101034b0d/src/tss2-esys/esys_crypto_ossl.c#L924
This is easy to workaround by hiding the case.
Are there good reasons why LibreSSL should implement this in the future?
I made WIP upstream PR here. tpm2-software/tpm2-tss#2380
The text was updated successfully, but these errors were encountered: