diff options
| author | tb <> | 2024-01-25 14:08:30 +0000 |
|---|---|---|
| committer | tb <> | 2024-01-25 14:08:30 +0000 |
| commit | 31775d73af76ef18e0ec9823ced68db3ce2bc611 (patch) | |
| tree | 2103872c395642abe04e0ded5ac057c62d1ca67e /src | |
| parent | 26fee542e65d530cdacb9282bf510602c1e2b5fd (diff) | |
| download | openbsd-31775d73af76ef18e0ec9823ced68db3ce2bc611.tar.gz openbsd-31775d73af76ef18e0ec9823ced68db3ce2bc611.tar.bz2 openbsd-31775d73af76ef18e0ec9823ced68db3ce2bc611.zip | |
p12_npas.c: Use slightly less awkward variable names
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/libcrypto/pkcs12/p12_npas.c | 42 |
1 files changed, 21 insertions, 21 deletions
diff --git a/src/lib/libcrypto/pkcs12/p12_npas.c b/src/lib/libcrypto/pkcs12/p12_npas.c index fc726f2b74..b90ffe3da9 100644 --- a/src/lib/libcrypto/pkcs12/p12_npas.c +++ b/src/lib/libcrypto/pkcs12/p12_npas.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: p12_npas.c,v 1.23 2024/01/25 13:32:49 tb Exp $ */ | 1 | /* $OpenBSD: p12_npas.c,v 1.24 2024/01/25 14:08:30 tb Exp $ */ |
| 2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL | 2 | /* Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL |
| 3 | * project 1999. | 3 | * project 1999. |
| 4 | */ | 4 | */ |
| @@ -79,39 +79,39 @@ static int alg_get(X509_ALGOR *alg, int *pnid, int *piter, int *psaltlen); | |||
| 79 | */ | 79 | */ |
| 80 | 80 | ||
| 81 | static int | 81 | static int |
| 82 | pkcs7_repack_data(PKCS7 *pkcs7, STACK_OF(PKCS7) *newsafes, const char *oldpass, | 82 | pkcs7_repack_data(PKCS7 *pkcs7, STACK_OF(PKCS7) *safes, const char *oldpass, |
| 83 | const char *newpass) | 83 | const char *newpass) |
| 84 | { | 84 | { |
| 85 | STACK_OF(PKCS12_SAFEBAG) *bags; | 85 | STACK_OF(PKCS12_SAFEBAG) *bags; |
| 86 | PKCS7 *newdata = NULL; | 86 | PKCS7 *data = NULL; |
| 87 | int ret = 0; | 87 | int ret = 0; |
| 88 | 88 | ||
| 89 | if ((bags = PKCS12_unpack_p7data(pkcs7)) == NULL) | 89 | if ((bags = PKCS12_unpack_p7data(pkcs7)) == NULL) |
| 90 | goto err; | 90 | goto err; |
| 91 | if (!newpass_bags(bags, oldpass, newpass)) | 91 | if (!newpass_bags(bags, oldpass, newpass)) |
| 92 | goto err; | 92 | goto err; |
| 93 | if ((newdata = PKCS12_pack_p7data(bags)) == NULL) | 93 | if ((data = PKCS12_pack_p7data(bags)) == NULL) |
| 94 | goto err; | 94 | goto err; |
| 95 | if (sk_PKCS7_push(newsafes, newdata) == 0) | 95 | if (sk_PKCS7_push(safes, data) == 0) |
| 96 | goto err; | 96 | goto err; |
| 97 | newdata = NULL; | 97 | data = NULL; |
| 98 | 98 | ||
| 99 | ret = 1; | 99 | ret = 1; |
| 100 | 100 | ||
| 101 | err: | 101 | err: |
| 102 | sk_PKCS12_SAFEBAG_pop_free(bags, PKCS12_SAFEBAG_free); | 102 | sk_PKCS12_SAFEBAG_pop_free(bags, PKCS12_SAFEBAG_free); |
| 103 | PKCS7_free(newdata); | 103 | PKCS7_free(data); |
| 104 | 104 | ||
| 105 | return ret; | 105 | return ret; |
| 106 | } | 106 | } |
| 107 | 107 | ||
| 108 | static int | 108 | static int |
| 109 | pkcs7_repack_encdata(PKCS7 *pkcs7, STACK_OF(PKCS7) *newsafes, const char *oldpass, | 109 | pkcs7_repack_encdata(PKCS7 *pkcs7, STACK_OF(PKCS7) *safes, const char *oldpass, |
| 110 | const char *newpass) | 110 | const char *newpass) |
| 111 | { | 111 | { |
| 112 | STACK_OF(PKCS12_SAFEBAG) *bags; | 112 | STACK_OF(PKCS12_SAFEBAG) *bags; |
| 113 | int pbe_nid = 0, pbe_iter = 0, pbe_saltlen = 0; | 113 | int pbe_nid = 0, pbe_iter = 0, pbe_saltlen = 0; |
| 114 | PKCS7 *newencdata = NULL; | 114 | PKCS7 *data = NULL; |
| 115 | int ret = 0; | 115 | int ret = 0; |
| 116 | 116 | ||
| 117 | if ((bags = PKCS12_unpack_p7encdata(pkcs7, oldpass, -1)) == NULL) | 117 | if ((bags = PKCS12_unpack_p7encdata(pkcs7, oldpass, -1)) == NULL) |
| @@ -121,24 +121,24 @@ pkcs7_repack_encdata(PKCS7 *pkcs7, STACK_OF(PKCS7) *newsafes, const char *oldpas | |||
| 121 | goto err; | 121 | goto err; |
| 122 | if (!newpass_bags(bags, oldpass, newpass)) | 122 | if (!newpass_bags(bags, oldpass, newpass)) |
| 123 | goto err; | 123 | goto err; |
| 124 | if ((newencdata = PKCS12_pack_p7encdata(pbe_nid, newpass, -1, NULL, | 124 | if ((data = PKCS12_pack_p7encdata(pbe_nid, newpass, -1, NULL, |
| 125 | pbe_saltlen, pbe_iter, bags)) == NULL) | 125 | pbe_saltlen, pbe_iter, bags)) == NULL) |
| 126 | goto err; | 126 | goto err; |
| 127 | if (!sk_PKCS7_push(newsafes, newencdata)) | 127 | if (!sk_PKCS7_push(safes, data)) |
| 128 | goto err; | 128 | goto err; |
| 129 | newencdata = NULL; | 129 | data = NULL; |
| 130 | 130 | ||
| 131 | ret = 1; | 131 | ret = 1; |
| 132 | 132 | ||
| 133 | err: | 133 | err: |
| 134 | sk_PKCS12_SAFEBAG_pop_free(bags, PKCS12_SAFEBAG_free); | 134 | sk_PKCS12_SAFEBAG_pop_free(bags, PKCS12_SAFEBAG_free); |
| 135 | PKCS7_free(newencdata); | 135 | PKCS7_free(data); |
| 136 | 136 | ||
| 137 | return ret; | 137 | return ret; |
| 138 | } | 138 | } |
| 139 | 139 | ||
| 140 | static int | 140 | static int |
| 141 | pkcs12_repack_authsafes(PKCS12 *pkcs12, STACK_OF(PKCS7) *newsafes, | 141 | pkcs12_repack_authsafes(PKCS12 *pkcs12, STACK_OF(PKCS7) *safes, |
| 142 | const char *newpass) | 142 | const char *newpass) |
| 143 | { | 143 | { |
| 144 | ASN1_OCTET_STRING *old_data; | 144 | ASN1_OCTET_STRING *old_data; |
| @@ -151,7 +151,7 @@ pkcs12_repack_authsafes(PKCS12 *pkcs12, STACK_OF(PKCS7) *newsafes, | |||
| 151 | goto err; | 151 | goto err; |
| 152 | if ((pkcs12->authsafes->d.data = ASN1_OCTET_STRING_new()) == NULL) | 152 | if ((pkcs12->authsafes->d.data = ASN1_OCTET_STRING_new()) == NULL) |
| 153 | goto err; | 153 | goto err; |
| 154 | if (!PKCS12_pack_authsafes(pkcs12, newsafes)) | 154 | if (!PKCS12_pack_authsafes(pkcs12, safes)) |
| 155 | goto err; | 155 | goto err; |
| 156 | if (!PKCS12_gen_mac(pkcs12, newpass, -1, mac, &maclen)) | 156 | if (!PKCS12_gen_mac(pkcs12, newpass, -1, mac, &maclen)) |
| 157 | goto err; | 157 | goto err; |
| @@ -183,7 +183,7 @@ pkcs12_repack_authsafes(PKCS12 *pkcs12, STACK_OF(PKCS7) *newsafes, | |||
| 183 | int | 183 | int |
| 184 | PKCS12_newpass(PKCS12 *pkcs12, const char *oldpass, const char *newpass) | 184 | PKCS12_newpass(PKCS12 *pkcs12, const char *oldpass, const char *newpass) |
| 185 | { | 185 | { |
| 186 | STACK_OF(PKCS7) *authsafes = NULL, *newsafes = NULL; | 186 | STACK_OF(PKCS7) *authsafes = NULL, *safes = NULL; |
| 187 | int i; | 187 | int i; |
| 188 | int ret = 0; | 188 | int ret = 0; |
| 189 | 189 | ||
| @@ -199,7 +199,7 @@ PKCS12_newpass(PKCS12 *pkcs12, const char *oldpass, const char *newpass) | |||
| 199 | 199 | ||
| 200 | if ((authsafes = PKCS12_unpack_authsafes(pkcs12)) == NULL) | 200 | if ((authsafes = PKCS12_unpack_authsafes(pkcs12)) == NULL) |
| 201 | goto err; | 201 | goto err; |
| 202 | if ((newsafes = sk_PKCS7_new_null()) == NULL) | 202 | if ((safes = sk_PKCS7_new_null()) == NULL) |
| 203 | goto err; | 203 | goto err; |
| 204 | 204 | ||
| 205 | for (i = 0; i < sk_PKCS7_num(authsafes); i++) { | 205 | for (i = 0; i < sk_PKCS7_num(authsafes); i++) { |
| @@ -207,24 +207,24 @@ PKCS12_newpass(PKCS12 *pkcs12, const char *oldpass, const char *newpass) | |||
| 207 | 207 | ||
| 208 | switch (OBJ_obj2nid(pkcs7->type)) { | 208 | switch (OBJ_obj2nid(pkcs7->type)) { |
| 209 | case NID_pkcs7_data: | 209 | case NID_pkcs7_data: |
| 210 | if (pkcs7_repack_data(pkcs7, newsafes, oldpass, newpass)) | 210 | if (pkcs7_repack_data(pkcs7, safes, oldpass, newpass)) |
| 211 | goto err; | 211 | goto err; |
| 212 | break; | 212 | break; |
| 213 | case NID_pkcs7_encrypted: | 213 | case NID_pkcs7_encrypted: |
| 214 | if (pkcs7_repack_encdata(pkcs7, newsafes, oldpass, newpass)) | 214 | if (pkcs7_repack_encdata(pkcs7, safes, oldpass, newpass)) |
| 215 | goto err; | 215 | goto err; |
| 216 | break; | 216 | break; |
| 217 | } | 217 | } |
| 218 | } | 218 | } |
| 219 | 219 | ||
| 220 | if (!pkcs12_repack_authsafes(pkcs12, newsafes, newpass)) | 220 | if (!pkcs12_repack_authsafes(pkcs12, safes, newpass)) |
| 221 | goto err; | 221 | goto err; |
| 222 | 222 | ||
| 223 | ret = 1; | 223 | ret = 1; |
| 224 | 224 | ||
| 225 | err: | 225 | err: |
| 226 | sk_PKCS7_pop_free(authsafes, PKCS7_free); | 226 | sk_PKCS7_pop_free(authsafes, PKCS7_free); |
| 227 | sk_PKCS7_pop_free(newsafes, PKCS7_free); | 227 | sk_PKCS7_pop_free(safes, PKCS7_free); |
| 228 | 228 | ||
| 229 | return ret; | 229 | return ret; |
| 230 | } | 230 | } |
