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

bio/bss_file.c: since VS2015 one can't tell apart own and "alien" FILE pointers, except for minimal std[in|out|err]. (1.0.2 backport) #2701

Closed
wants to merge 1 commit into from
Closed

bio/bss_file.c: since VS2015 one can't tell apart own and "alien" FILE pointers, except for minimal std[in|out|err]. (1.0.2 backport) #2701

wants to merge 1 commit into from

Conversation

levitte
Copy link
Member

@levitte levitte commented Feb 21, 2017

Description of change

Fixes #2404, for 1.0.2

@levitte
bio/bss_file.c: since VS2015 one can't tell apart own and "alien" FILE
3de6834 
pointers, except for minimal std[in|out|err].

Reviewed-by: Richard Levitte <levitte@openssl.org>
(cherry picked from commit 800b299)
@levitte levitte mentioned this pull request Feb 21, 2017
Closed
@levitte
Copy link
Member Author

levitte commented Feb 21, 2017

Reviewer requested to approve this backport.

@richsalz richsalz added branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch reviewed labels Feb 21, 2017
levitte pushed a commit that referenced this pull request Feb 23, 2017
@levitte
bio/bss_file.c: since VS2015 one can't tell apart own and "alien" FILE
56fbd5c 
pointers, except for minimal std[in|out|err].

(cherry picked from commit 800b299)
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from #2701)
@levitte
Copy link
Member Author

levitte commented Feb 23, 2017

( @richsalz, FYI, setting labels doesn't seem to activate any notification whatsoever. Approval does... )

Merged into 1.0.2. 56fbd5c

@levitte levitte closed this Feb 23, 2017
@sunnyap sunnyap mentioned this pull request Feb 23, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
branch: 1.0.2 Merge to OpenSSL_1_0_2-stable branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@levitte @richsalz