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

Potential null pointer reference #19428

Closed
yangyangtiantianlonglong opened this issue Oct 18, 2022 · 0 comments
Closed

Potential null pointer reference #19428

yangyangtiantianlonglong opened this issue Oct 18, 2022 · 0 comments
Labels
branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch triaged: bug The issue/pr is/fixes a bug

Comments

@yangyangtiantianlonglong
Copy link
Contributor

openssl/test/ectest.c

Lines 2724 to 2738 in 846975f

if (!TEST_ptr(ctx = BN_CTX_new()))
return 0;
if (!TEST_ptr(group = EC_GROUP_new_by_curve_name(nid)))
goto err;
is_prime = EC_GROUP_get_field_type(group) == NID_X9_62_prime_field;
#ifdef OPENSSL_NO_EC2M
if (!is_prime) {
ret = TEST_skip("binary curves not supported in this build");
goto err;
}
#endif
BN_CTX_start(ctx);

test/ectest.c:2728 and 2734 may cause BN_CTX_end() -->BN_STACK_pop() coredump.

@yangyangtiantianlonglong yangyangtiantianlonglong added the issue: bug report The issue was opened to report a bug label Oct 18, 2022
@t8m t8m added triaged: bug The issue/pr is/fixes a bug branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch and removed issue: bug report The issue was opened to report a bug labels Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch: master Merge to master branch branch: 3.0 Merge to openssl-3.0 branch triaged: bug The issue/pr is/fixes a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants