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

[Backport 2.16] NetBSD/OpenBSD symbol availability fix #3602

Merged
merged 1 commit into from
Nov 12, 2020
Merged

[Backport 2.16] NetBSD/OpenBSD symbol availability fix #3602

merged 1 commit into from
Nov 12, 2020

Conversation

gufe44
Copy link
Contributor

@gufe44 gufe44 commented Aug 23, 2020

This is a backport of #3571.

Edit: added OpenBSD counterpart (see stdlib.h#L316-L319 and cdefs.h#L392-L395)

@gufe44 gufe44 changed the title [Backport 2.16] NetBSD symbol availability fix [Backport 2.16] NetBSD/OpenBSD symbol availability fix Aug 29, 2020
@daverodgman daverodgman self-assigned this Sep 10, 2020
@daverodgman daverodgman added needs-ci Needs to pass CI tests needs-review Every commit must be reviewed by at least two team members, labels Sep 10, 2020
@yanesca yanesca added Community needs-reviewer This PR needs someone to pick it up for review labels Oct 2, 2020
@gilles-peskine-arm gilles-peskine-arm removed the needs-ci Needs to pass CI tests label Oct 6, 2020
@daverodgman daverodgman self-requested a review October 6, 2020 14:41
Copy link
Contributor

@daverodgman daverodgman left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

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

As in #3656 (comment): _xxx_SOURCE should not be redefined if it's already defined.

@gilles-peskine-arm gilles-peskine-arm added needs-work and removed needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review labels Oct 13, 2020
@gufe44
Copy link
Contributor Author

gufe44 commented Oct 30, 2020

Unfortunately, for personal reasons I won't be able to continue to work on the PR. You can close the PR now and resurrect the branch at any time.

@gilles-peskine-arm gilles-peskine-arm merged commit a3e5753 into Mbed-TLS:mbedtls-2.16 Nov 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants