Skip to content

Remove redundant ASN1_INTEGER_set call#6984

Closed
ericwb wants to merge 1 commit intoopenssl:masterfrom
ericwb:rm_redundant
Closed

Remove redundant ASN1_INTEGER_set call#6984
ericwb wants to merge 1 commit intoopenssl:masterfrom
ericwb:rm_redundant

Conversation

@ericwb
Copy link
Contributor

@ericwb ericwb commented Aug 16, 2018

This trivial patch removes a duplicated call to ASN1_INTEGER_set.

Fixes Issue #6977

Signed-off-by: Eric Brown [email protected]

This trivial patch removes a duplicated call to ASN1_INTEGER_set.

Fixes Issue #6977

Signed-off-by: Eric Brown <[email protected]>
@dot-asm dot-asm added the approval: done This pull request has the required number of approvals label Aug 16, 2018
@InfoHunter
Copy link
Member

Hi @levitte and @dot-asm , this PR has been approved 7 days ago, would you guys like me to merge this? :-)

@paulidale paulidale added branch: 1.0.2 Applies to OpenSSL_1_0_2-stable branch (EOL) 1.1.0 branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) labels Sep 2, 2018
levitte pushed a commit that referenced this pull request Sep 2, 2018
This trivial patch removes a duplicated call to ASN1_INTEGER_set.

Fixes Issue #6977

Signed-off-by: Eric Brown <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Andy Polyakov <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from #6984)
levitte pushed a commit that referenced this pull request Sep 2, 2018
This trivial patch removes a duplicated call to ASN1_INTEGER_set.

Fixes Issue #6977

Signed-off-by: Eric Brown <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Andy Polyakov <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from #6984)

(cherry picked from commit 59701e6)
levitte pushed a commit that referenced this pull request Sep 2, 2018
This trivial patch removes a duplicated call to ASN1_INTEGER_set.

Fixes Issue #6977

Signed-off-by: Eric Brown <[email protected]>

Reviewed-by: Richard Levitte <[email protected]>
Reviewed-by: Andy Polyakov <[email protected]>
Reviewed-by: Paul Dale <[email protected]>
(Merged from #6984)

(cherry picked from commit 59701e6)
@paulidale
Copy link
Contributor

Merged to 1.0.2, 1.1.0 and master.

Thanks.

@paulidale paulidale closed this Sep 2, 2018
@ericwb ericwb deleted the rm_redundant branch September 2, 2018 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: 1.0.2 Applies to OpenSSL_1_0_2-stable branch (EOL) branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants