Skip to content

Comments

Add const to 'ppin' function parameter in d2i_X509.pod and d2i_DHparams.pod#12205

Closed
olszomal wants to merge 1 commit intoopenssl:masterfrom
olszomal:d2i_TYPE_manual
Closed

Add const to 'ppin' function parameter in d2i_X509.pod and d2i_DHparams.pod#12205
olszomal wants to merge 1 commit intoopenssl:masterfrom
olszomal:d2i_TYPE_manual

Conversation

@olszomal
Copy link
Contributor

The function parameter ppin should be declared as the const unsigned char type according to this declaration

# define DECLARE_ASN1_ENCODE_FUNCTIONS_only(type, name) \
        type *d2i_##name(type **a, const unsigned char **in, long len); 

[skip ci]
CLA: trivial

@kroeckx kroeckx added approval: review pending This pull request needs review by a committer branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) branch: master Applies to master branch labels Sep 16, 2020
@mattcaswell mattcaswell added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Sep 17, 2020
@openssl-machine openssl-machine removed the approval: done This pull request has the required number of approvals label Sep 18, 2020
@openssl-machine
Copy link
Collaborator

This pull request is ready to merge

@openssl-machine openssl-machine added the approval: ready to merge The 24 hour grace period has passed, ready to merge label Sep 18, 2020
@kroeckx kroeckx added the cla: trivial One of the commits is marked as 'CLA: trivial' label Sep 20, 2020
@kroeckx
Copy link
Member

kroeckx commented Sep 20, 2020

matt: Do you agree this is trivial?

@mattcaswell
Copy link
Member

matt: Do you agree this is trivial?

Yes

openssl-machine pushed a commit that referenced this pull request Sep 21, 2020
CLA: trivial

Reviewed-by: Kurt Roeckx <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
GH: #12205
openssl-machine pushed a commit that referenced this pull request Sep 21, 2020
CLA: trivial

Reviewed-by: Kurt Roeckx <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
GH: #12205
(cherry picked from commit 434343f)
@kroeckx
Copy link
Member

kroeckx commented Sep 21, 2020

merged

@kroeckx kroeckx closed this Sep 21, 2020
@olszomal olszomal deleted the d2i_TYPE_manual branch July 11, 2022 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) cla: trivial One of the commits is marked as 'CLA: trivial'

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants