Skip to content

Comments

Fix build issues with no-dh, no-dsa and no-ec#2192

Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-build-issues-20170108-1
Closed

Fix build issues with no-dh, no-dsa and no-ec#2192
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-build-issues-20170108-1

Conversation

@levitte
Copy link
Member

@levitte levitte commented Jan 8, 2017

Checklist
  • CLA is signed
Description of change

fuzz/server.c Outdated
Copy link
Member

Choose a reason for hiding this comment

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

I think the NO_DSA doesn't make sense at this line (and some others), I don't think the EC(DSA) code shares anything with DSA code.

Copy link
Member Author

Choose a reason for hiding this comment

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

You're right. Fixing

@levitte levitte force-pushed the fix-build-issues-20170108-1 branch from f9c67f1 to 15a19c3 Compare January 9, 2017 00:11
@levitte
Copy link
Member Author

levitte commented Jan 9, 2017

Please have a look now, @kroeckx

@levitte
Copy link
Member Author

levitte commented Jan 9, 2017

Merged

@levitte levitte closed this Jan 9, 2017
levitte added a commit that referenced this pull request Jan 9, 2017
busterb pushed a commit to libressl/fuzz that referenced this pull request Jan 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants