summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authortb <>2024-01-25 14:08:30 +0000
committertb <>2024-01-25 14:08:30 +0000
commit1d89cb1dff32f17cd8cb8c15fa2a56ad4088bccd (patch)
tree2103872c395642abe04e0ded5ac057c62d1ca67e
parent0c15b6210877e84c0a155c41b8c76f1a978d3378 (diff)
downloadopenbsd-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.c42
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
81static int 81static int
82pkcs7_repack_data(PKCS7 *pkcs7, STACK_OF(PKCS7) *newsafes, const char *oldpass, 82pkcs7_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
108static int 108static int
109pkcs7_repack_encdata(PKCS7 *pkcs7, STACK_OF(PKCS7) *newsafes, const char *oldpass, 109pkcs7_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
140static int 140static int
141pkcs12_repack_authsafes(PKCS12 *pkcs12, STACK_OF(PKCS7) *newsafes, 141pkcs12_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,
183int 183int
184PKCS12_newpass(PKCS12 *pkcs12, const char *oldpass, const char *newpass) 184PKCS12_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}