diff options
author | tb <> | 2024-01-25 14:08:30 +0000 |
---|---|---|
committer | tb <> | 2024-01-25 14:08:30 +0000 |
commit | 1d89cb1dff32f17cd8cb8c15fa2a56ad4088bccd (patch) | |
tree | 2103872c395642abe04e0ded5ac057c62d1ca67e | |
parent | 0c15b6210877e84c0a155c41b8c76f1a978d3378 (diff) | |
download | openbsd-1d89cb1dff32f17cd8cb8c15fa2a56ad4088bccd.tar.gz openbsd-1d89cb1dff32f17cd8cb8c15fa2a56ad4088bccd.tar.bz2 openbsd-1d89cb1dff32f17cd8cb8c15fa2a56ad4088bccd.zip |
p12_npas.c: Use slightly less awkward variable names
-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 | } |