ok bcook@
On Fri, May 18, 2018 at 9:13 AM, Theo Buehler <t...@theobuehler.org> wrote:
> On Fri, May 18, 2018 at 04:03:16PM +0200, Theo Buehler wrote:
> > Simple diff that adds const qualifiers to the X509_CRL *x and
> > ASN1_OBJECT *obj arguments of X509_CRL_get_ext_count(3),
> > X509_CRL_get_ext_by_NID(3), X509_CRL_get_ext_by_OBJ(3),
> > X509_CRL_get_ext_by_critical(3), X509_CRL_get_ext(3),
> > X509_CRL_get_ext_d2i(3).
>
> I forgot to mention two things. First, this continues my working through
> the big diff that was in sthen's bulk last week. Second, please don't
> request whitespace changes here. I've already wasted enough time on
> this.
>
> >
> > Index: lib/libcrypto/x509/x509.h
> > ===================================================================
> > RCS file: /var/cvs/src/lib/libcrypto/x509/x509.h,v
> > retrieving revision 1.49
> > diff -u -p -r1.49 x509.h
> > --- lib/libcrypto/x509/x509.h 13 May 2018 10:36:35 -0000 1.49
> > +++ lib/libcrypto/x509/x509.h 18 May 2018 13:54:54 -0000
> > @@ -1149,16 +1149,20 @@ void * X509_get_ext_d2i(X509 *x, int nid
> > int X509_add1_ext_i2d(X509 *x, int nid, void *value, int crit,
> > unsigned long
> flags);
> >
> > -int X509_CRL_get_ext_count(X509_CRL *x);
> > -int X509_CRL_get_ext_by_NID(X509_CRL *x, int nid, int
> lastpos);
> > -int X509_CRL_get_ext_by_OBJ(X509_CRL *x,ASN1_OBJECT *obj,int
> lastpos);
> > -int X509_CRL_get_ext_by_critical(X509_CRL *x, int crit, int
> lastpos);
> > -X509_EXTENSION *X509_CRL_get_ext(X509_CRL *x, int loc);
> > +int X509_CRL_get_ext_count(const X509_CRL *x);
> > +int X509_CRL_get_ext_by_NID(const X509_CRL *x, int nid,
> > + int lastpos);
> > +int X509_CRL_get_ext_by_OBJ(const X509_CRL *x,
> > + const ASN1_OBJECT *obj, int lastpos);
> > +int X509_CRL_get_ext_by_critical(const X509_CRL *x, int crit,
> > + int lastpos);
> > +X509_EXTENSION *X509_CRL_get_ext(const X509_CRL *x, int loc);
> > X509_EXTENSION *X509_CRL_delete_ext(X509_CRL *x, int loc);
> > int X509_CRL_add_ext(X509_CRL *x, X509_EXTENSION *ex, int loc);
> > -void * X509_CRL_get_ext_d2i(X509_CRL *x, int nid, int *crit, int
> *idx);
> > -int X509_CRL_add1_ext_i2d(X509_CRL *x, int nid, void *value,
> int crit,
> > - unsigned long
> flags);
> > +void * X509_CRL_get_ext_d2i(const X509_CRL *x, int nid, int *crit,
> > + int *idx);
> > +int X509_CRL_add1_ext_i2d(X509_CRL *x, int nid, void *value,
> > + int crit, unsigned long flags);
> >
> > int X509_REVOKED_get_ext_count(X509_REVOKED *x);
> > int X509_REVOKED_get_ext_by_NID(X509_REVOKED *x, int nid, int
> lastpos);
> > Index: lib/libcrypto/x509/x509_ext.c
> > ===================================================================
> > RCS file: /var/cvs/src/lib/libcrypto/x509/x509_ext.c,v
> > retrieving revision 1.9
> > diff -u -p -r1.9 x509_ext.c
> > --- lib/libcrypto/x509/x509_ext.c 10 Feb 2015 08:33:10 -0000 1.9
> > +++ lib/libcrypto/x509/x509_ext.c 18 May 2018 13:54:54 -0000
> > @@ -66,31 +66,31 @@
> > #include <openssl/x509v3.h>
> >
> > int
> > -X509_CRL_get_ext_count(X509_CRL *x)
> > +X509_CRL_get_ext_count(const X509_CRL *x)
> > {
> > return (X509v3_get_ext_count(x->crl->extensions));
> > }
> >
> > int
> > -X509_CRL_get_ext_by_NID(X509_CRL *x, int nid, int lastpos)
> > +X509_CRL_get_ext_by_NID(const X509_CRL *x, int nid, int lastpos)
> > {
> > return (X509v3_get_ext_by_NID(x->crl->extensions, nid, lastpos));
> > }
> >
> > int
> > -X509_CRL_get_ext_by_OBJ(X509_CRL *x, ASN1_OBJECT *obj, int lastpos)
> > +X509_CRL_get_ext_by_OBJ(const X509_CRL *x, const ASN1_OBJECT *obj, int
> lastpos)
> > {
> > return (X509v3_get_ext_by_OBJ(x->crl->extensions, obj, lastpos));
> > }
> >
> > int
> > -X509_CRL_get_ext_by_critical(X509_CRL *x, int crit, int lastpos)
> > +X509_CRL_get_ext_by_critical(const X509_CRL *x, int crit, int lastpos)
> > {
> > return (X509v3_get_ext_by_critical(x->crl->extensions, crit,
> lastpos));
> > }
> >
> > X509_EXTENSION *
> > -X509_CRL_get_ext(X509_CRL *x, int loc)
> > +X509_CRL_get_ext(const X509_CRL *x, int loc)
> > {
> > return (X509v3_get_ext(x->crl->extensions, loc));
> > }
> > @@ -102,7 +102,7 @@ X509_CRL_delete_ext(X509_CRL *x, int loc
> > }
> >
> > void *
> > -X509_CRL_get_ext_d2i(X509_CRL *x, int nid, int *crit, int *idx)
> > +X509_CRL_get_ext_d2i(const X509_CRL *x, int nid, int *crit, int *idx)
> > {
> > return X509V3_get_d2i(x->crl->extensions, nid, crit, idx);
> > }
> >
>
