diff options
author | Justin Blanchard <uncombedcoconut@gmail.com> | 2014-08-18 11:01:15 -0400 |
---|---|---|
committer | Rich Salz <rsalz@akamai.com> | 2014-08-18 11:49:16 -0400 |
commit | f756fb430eb8f5f70696f174460eb90740b318f7 (patch) | |
tree | ce3c86c0c6717e30193a3ba15d176e390d322707 /crypto | |
parent | defe438d472bbef4d2bc5e4893560bc4eae43760 (diff) | |
download | openssl-f756fb430eb8f5f70696f174460eb90740b318f7.zip openssl-f756fb430eb8f5f70696f174460eb90740b318f7.tar.gz openssl-f756fb430eb8f5f70696f174460eb90740b318f7.tar.bz2 |
RT1815: More const'ness improvements
Add a dozen more const declarations where appropriate.
These are from Justin; while adding his patch, I noticed
ASN1_BIT_STRING_check could be fixed, too.
Reviewed-by: Dr. Stephen Henson <steve@openssl.org>
Diffstat (limited to 'crypto')
-rw-r--r-- | crypto/asn1/a_bitstr.c | 6 | ||||
-rw-r--r-- | crypto/asn1/asn1.h | 6 | ||||
-rw-r--r-- | crypto/asn1/charmap.pl | 2 | ||||
-rw-r--r-- | crypto/asn1/tasn_prn.c | 4 | ||||
-rw-r--r-- | crypto/conf/conf_def.h | 6 | ||||
-rw-r--r-- | crypto/conf/keysets.pl | 6 | ||||
-rw-r--r-- | crypto/engine/eng_openssl.c | 8 | ||||
-rw-r--r-- | crypto/evp/openbsd_hw.c | 2 | ||||
-rw-r--r-- | crypto/ts/ts_rsp_print.c | 10 |
9 files changed, 25 insertions, 25 deletions
diff --git a/crypto/asn1/a_bitstr.c b/crypto/asn1/a_bitstr.c index 3417996..0cb899f 100644 --- a/crypto/asn1/a_bitstr.c +++ b/crypto/asn1/a_bitstr.c @@ -212,7 +212,7 @@ int ASN1_BIT_STRING_set_bit(ASN1_BIT_STRING *a, int n, int value) return(1); } -int ASN1_BIT_STRING_get_bit(ASN1_BIT_STRING *a, int n) +int ASN1_BIT_STRING_get_bit(const ASN1_BIT_STRING *a, int n) { int w,v; @@ -229,8 +229,8 @@ int ASN1_BIT_STRING_get_bit(ASN1_BIT_STRING *a, int n) * which is not specified in 'flags', 1 otherwise. * 'len' is the length of 'flags'. */ -int ASN1_BIT_STRING_check(ASN1_BIT_STRING *a, - unsigned char *flags, int flags_len) +int ASN1_BIT_STRING_check(const ASN1_BIT_STRING *a, + const unsigned char *flags, int flags_len) { int i, ok; /* Check if there is one bit set at all. */ diff --git a/crypto/asn1/asn1.h b/crypto/asn1/asn1.h index d6342d0..925fdba 100644 --- a/crypto/asn1/asn1.h +++ b/crypto/asn1/asn1.h @@ -819,9 +819,9 @@ ASN1_BIT_STRING *c2i_ASN1_BIT_STRING(ASN1_BIT_STRING **a,const unsigned char **p int ASN1_BIT_STRING_set(ASN1_BIT_STRING *a, unsigned char *d, int length ); int ASN1_BIT_STRING_set_bit(ASN1_BIT_STRING *a, int n, int value); -int ASN1_BIT_STRING_get_bit(ASN1_BIT_STRING *a, int n); -int ASN1_BIT_STRING_check(ASN1_BIT_STRING *a, - unsigned char *flags, int flags_len); +int ASN1_BIT_STRING_get_bit(const ASN1_BIT_STRING *a, int n); +int ASN1_BIT_STRING_check(const ASN1_BIT_STRING *a, + const unsigned char *flags, int flags_len); #ifndef OPENSSL_NO_BIO int ASN1_BIT_STRING_name_print(BIO *out, ASN1_BIT_STRING *bs, diff --git a/crypto/asn1/charmap.pl b/crypto/asn1/charmap.pl index 25ebf2c..878504f 100644 --- a/crypto/asn1/charmap.pl +++ b/crypto/asn1/charmap.pl @@ -71,7 +71,7 @@ print <<EOF; * Mask of various character properties */ -static unsigned char char_type[] = { +static const unsigned char char_type[] = { EOF for($i = 0; $i < 128; $i++) { diff --git a/crypto/asn1/tasn_prn.c b/crypto/asn1/tasn_prn.c index 542a091..424d395 100644 --- a/crypto/asn1/tasn_prn.c +++ b/crypto/asn1/tasn_prn.c @@ -401,8 +401,8 @@ static int asn1_print_fsname(BIO *out, int indent, const char *fname, const char *sname, const ASN1_PCTX *pctx) { - static char spaces[] = " "; - const int nspaces = sizeof(spaces) - 1; + static const char spaces[] = " "; + static const int nspaces = sizeof(spaces) - 1; #if 0 if (!sname && !fname) diff --git a/crypto/conf/conf_def.h b/crypto/conf/conf_def.h index 92a7d8a..f26acd7 100644 --- a/crypto/conf/conf_def.h +++ b/crypto/conf/conf_def.h @@ -77,7 +77,7 @@ #define CONF_ALPHA_NUMERIC_PUNCT (CONF_ALPHA|CONF_NUMBER|CONF_UNDER| \ CONF_PUNCTUATION) -#define KEYTYPES(c) ((unsigned short *)((c)->meth_data)) +#define KEYTYPES(c) ((const unsigned short *)((c)->meth_data)) #ifndef CHARSET_EBCDIC #define IS_COMMENT(c,a) (KEYTYPES(c)[(a)&0xff]&CONF_COMMENT) #define IS_FCOMMENT(c,a) (KEYTYPES(c)[(a)&0xff]&CONF_FCOMMENT) @@ -108,7 +108,7 @@ #define IS_HIGHBIT(c,a) (KEYTYPES(c)[os_toascii[a]&0xff]&CONF_HIGHBIT) #endif /*CHARSET_EBCDIC*/ -static unsigned short CONF_type_default[256]={ +static const unsigned short CONF_type_default[256]={ 0x0008,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000, 0x0000,0x0010,0x0010,0x0000,0x0000,0x0010,0x0000,0x0000, 0x0000,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000, @@ -143,7 +143,7 @@ static unsigned short CONF_type_default[256]={ 0x1000,0x1000,0x1000,0x1000,0x1000,0x1000,0x1000,0x1000, }; -static unsigned short CONF_type_win32[256]={ +static const unsigned short CONF_type_win32[256]={ 0x0008,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000, 0x0000,0x0010,0x0010,0x0000,0x0000,0x0010,0x0000,0x0000, 0x0000,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000,0x0000, diff --git a/crypto/conf/keysets.pl b/crypto/conf/keysets.pl index 50ed67f..a9baca4 100644 --- a/crypto/conf/keysets.pl +++ b/crypto/conf/keysets.pl @@ -131,7 +131,7 @@ print <<"EOF"; #define CONF_ALPHA_NUMERIC_PUNCT (CONF_ALPHA|CONF_NUMBER|CONF_UNDER| \\ CONF_PUNCTUATION) -#define KEYTYPES(c) ((unsigned short *)((c)->meth_data)) +#define KEYTYPES(c) ((const unsigned short *)((c)->meth_data)) #ifndef CHARSET_EBCDIC #define IS_COMMENT(c,a) (KEYTYPES(c)[(a)&0xff]&CONF_COMMENT) #define IS_FCOMMENT(c,a) (KEYTYPES(c)[(a)&0xff]&CONF_FCOMMENT) @@ -164,7 +164,7 @@ print <<"EOF"; EOF -print "static unsigned short CONF_type_default[256]={"; +print "static const unsigned short CONF_type_default[256]={"; for ($i=0; $i<256; $i++) { @@ -174,7 +174,7 @@ for ($i=0; $i<256; $i++) print "\n\t};\n\n"; -print "static unsigned short CONF_type_win32[256]={"; +print "static const unsigned short CONF_type_win32[256]={"; for ($i=0; $i<256; $i++) { diff --git a/crypto/engine/eng_openssl.c b/crypto/engine/eng_openssl.c index 3bdb779..db055a6 100644 --- a/crypto/engine/eng_openssl.c +++ b/crypto/engine/eng_openssl.c @@ -234,8 +234,8 @@ IMPLEMENT_DYNAMIC_BIND_FN(bind_fn) */ #include <openssl/rc4.h> #define TEST_RC4_KEY_SIZE 16 -static int test_cipher_nids[] = {NID_rc4,NID_rc4_40}; -static int test_cipher_nids_number = 2; +static const int test_cipher_nids[] = {NID_rc4,NID_rc4_40}; +static const int test_cipher_nids_number = 2; typedef struct { unsigned char key[TEST_RC4_KEY_SIZE]; RC4_KEY ks; @@ -319,8 +319,8 @@ static int openssl_ciphers(ENGINE *e, const EVP_CIPHER **cipher, #ifdef TEST_ENG_OPENSSL_SHA /* Much the same sort of comment as for TEST_ENG_OPENSSL_RC4 */ #include <openssl/sha.h> -static int test_digest_nids[] = {NID_sha1}; -static int test_digest_nids_number = 1; +static const int test_digest_nids[] = {NID_sha1}; +static const int test_digest_nids_number = 1; static int test_sha1_init(EVP_MD_CTX *ctx) { #ifdef TEST_ENG_OPENSSL_SHA_P_INIT diff --git a/crypto/evp/openbsd_hw.c b/crypto/evp/openbsd_hw.c index b746c46..e5252e2 100644 --- a/crypto/evp/openbsd_hw.c +++ b/crypto/evp/openbsd_hw.c @@ -326,7 +326,7 @@ static int dev_crypto_md5_init(EVP_MD_CTX *ctx) static int do_digest(int ses,unsigned char *md,const void *data,int len) { struct crypt_op cryp; - static unsigned char md5zero[16]= + static const unsigned char md5zero[16]= { 0xd4,0x1d,0x8c,0xd9,0x8f,0x00,0xb2,0x04, 0xe9,0x80,0x09,0x98,0xec,0xf8,0x42,0x7e diff --git a/crypto/ts/ts_rsp_print.c b/crypto/ts/ts_rsp_print.c index 2106251..4657f87 100644 --- a/crypto/ts/ts_rsp_print.c +++ b/crypto/ts/ts_rsp_print.c @@ -71,8 +71,8 @@ struct status_map_st /* Local function declarations. */ -static int TS_status_map_print(BIO *bio, struct status_map_st *a, - ASN1_BIT_STRING *v); +static int TS_status_map_print(BIO *bio, const struct status_map_st *a, + const ASN1_BIT_STRING *v); static int TS_ACCURACY_print_bio(BIO *bio, const TS_ACCURACY *accuracy); /* Function definitions. */ @@ -105,7 +105,7 @@ int TS_STATUS_INFO_print_bio(BIO *bio, TS_STATUS_INFO *a) "Revocation warning.", "Revoked." }; - static struct status_map_st failure_map[] = + static const struct status_map_st failure_map[] = { { TS_INFO_BAD_ALG, "unrecognized or unsupported algorithm identifier" }, @@ -162,8 +162,8 @@ int TS_STATUS_INFO_print_bio(BIO *bio, TS_STATUS_INFO *a) return 1; } -static int TS_status_map_print(BIO *bio, struct status_map_st *a, - ASN1_BIT_STRING *v) +static int TS_status_map_print(BIO *bio, const struct status_map_st *a, + const ASN1_BIT_STRING *v) { int lines = 0; |