Skip to content

Commit 3b4cf7b

Browse files
committed
bcrypt: Rearrange parameters of bcrypt_ctx_t related methods so they always
start with the algorithm state following with the context. * subversion/libsvn_subr/checksum_bcrypt.c (bcrypt_ctx_init, bcrypt_ctx_final): Update declarations. (bcrypt_ctx_update, bcrypt_checksum, svn_checksum__md5_ctx_final, svn_checksum__sha1_ctx_final): Update usages. git-svn-id: https://svn.apache.org/repos/asf/subversion/trunk@1931090 13f79535-47bb-0310-9956-ffa450edef68
1 parent 002d044 commit 3b4cf7b

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

subversion/libsvn_subr/checksum_bcrypt.c

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,8 @@ bcrypt_ctx_cleanup(void *data)
109109
}
110110

111111
static svn_error_t *
112-
bcrypt_ctx_init(bcrypt_ctx_t *ctx,
113-
algorithm_state_t *algorithm)
112+
bcrypt_ctx_init(algorithm_state_t *algorithm,
113+
bcrypt_ctx_t *ctx)
114114
{
115115
BCRYPT_HASH_HANDLE handle;
116116

@@ -137,7 +137,7 @@ bcrypt_ctx_update(algorithm_state_t *algorithm,
137137
SVN_ERR_ASSERT(len <= ULONG_MAX);
138138

139139
if (! ctx->handle)
140-
SVN_ERR(bcrypt_ctx_init(ctx, algorithm));
140+
SVN_ERR(bcrypt_ctx_init(algorithm, ctx));
141141

142142
SVN_ERR(handle_error(BCryptHashData(ctx->handle,
143143
(PUCHAR) data,
@@ -149,11 +149,11 @@ bcrypt_ctx_update(algorithm_state_t *algorithm,
149149

150150
static svn_error_t *
151151
bcrypt_ctx_final(algorithm_state_t *algorithm,
152-
unsigned char *digest,
153-
bcrypt_ctx_t *ctx)
152+
bcrypt_ctx_t *ctx,
153+
unsigned char *digest)
154154
{
155155
if (! ctx->handle)
156-
SVN_ERR(bcrypt_ctx_init(ctx, algorithm));
156+
SVN_ERR(bcrypt_ctx_init(algorithm, ctx));
157157

158158
SVN_ERR(handle_error(BCryptFinishHash(ctx->handle,
159159
(PUCHAR) digest,
@@ -179,7 +179,7 @@ bcrypt_checksum(algorithm_state_t *algorithm,
179179
bcrypt_ctx_t bcrypt_ctx;
180180
svn_error_t *err = SVN_NO_ERROR;
181181

182-
SVN_ERR(bcrypt_ctx_init(&bcrypt_ctx, algorithm));
182+
SVN_ERR(bcrypt_ctx_init(algorithm, &bcrypt_ctx));
183183

184184
err = bcrypt_ctx_update(algorithm, &bcrypt_ctx, data, len);
185185
if (err)
@@ -188,7 +188,7 @@ bcrypt_checksum(algorithm_state_t *algorithm,
188188
return err;
189189
}
190190

191-
SVN_ERR(bcrypt_ctx_final(algorithm, digest, &bcrypt_ctx));
191+
SVN_ERR(bcrypt_ctx_final(algorithm, &bcrypt_ctx, digest));
192192

193193
bcrypt_ctx_cleanup(&bcrypt_ctx);
194194
return err;
@@ -238,7 +238,8 @@ svn_error_t *
238238
svn_checksum__md5_ctx_final(unsigned char *digest,
239239
svn_checksum__md5_ctx_t *ctx)
240240
{
241-
return svn_error_trace(bcrypt_ctx_final(&md5, digest, &ctx->bcrypt_ctx));
241+
return svn_error_trace(bcrypt_ctx_final(&md5, &ctx->bcrypt_ctx,
242+
digest));
242243
}
243244

244245

@@ -285,7 +286,8 @@ svn_error_t *
285286
svn_checksum__sha1_ctx_final(unsigned char *digest,
286287
svn_checksum__sha1_ctx_t *ctx)
287288
{
288-
return svn_error_trace(bcrypt_ctx_final(&sha1, digest, &ctx->bcrypt_ctx));
289+
return svn_error_trace(bcrypt_ctx_final(&sha1, &ctx->bcrypt_ctx,
290+
digest));
289291
}
290292

291293
#endif /* SVN_CHECKSUM_BACKEND_APR */

0 commit comments

Comments
 (0)