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.

Sign up for GitHub

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

Jump to bottom

evp_test: when function and reason strings aren't available, just skip #2093

Closed
wants to merge 1 commit into from
Closed

evp_test: when function and reason strings aren't available, just skip #2093

wants to merge 1 commit into from

Conversation

levitte
Copy link
Member

@levitte levitte commented Dec 16, 2016

Checklist
  • CLA is signed
Description of change

When configuring no-autoerrinit, test/evp_test segfaults when trying to check error strings.

@levitte levitte added the branch: master Merge to master branch label Dec 16, 2016
richsalz
richsalz approved these changes Dec 16, 2016
View reviewed changes
Copy link
Contributor

@richsalz richsalz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and for 1.1.0 too

@levitte
Copy link
Member Author

levitte commented Dec 16, 2016

and for 1.1.0 too

Ah, hadn't noticed that code had been backported. Good, good, will do.

levitte added a commit that referenced this pull request Dec 16, 2016
@levitte
evp_test: when function and reason strings aren't available, just skip
cd3fe0e 
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #2093)
@levitte levitte added the 1.1.0 label Dec 16, 2016
levitte added a commit that referenced this pull request Dec 16, 2016
@levitte
evp_test: when function and reason strings aren't available, just skip
5769b54 
Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from #2093)
(cherry picked from commit cd3fe0e)
@levitte
Copy link
Member Author

levitte commented Dec 16, 2016

Merged

@levitte levitte closed this Dec 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richsalz richsalz richsalz approved these changes

@snhenson snhenson Awaiting requested review from snhenson

Assignees
No one assigned
Labels
branch: master Merge to master branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@levitte @richsalz