svn commit: r1931081 - subversion/trunk/subversion/libsvn_subr

[email protected]
Newsgroups gmane.comp.version-control.subversion.svn
Message-ID <176748342521.2374145.6572192729495954978@svn02-us-east.apache.org>
Author: rinrab
Date: Sat Jan  3 23:37:04 2026
New Revision: 1931081

Log:
bcrypt: Factor-out bcrypt_ctx_reset() function.

* subversion/libsvn_subr/checksum_bcrypt.c
  (bcrypt_ctx_reset): New method.
  (svn_checksum__md5_ctx_reset,
   svn_checksum__sha1_ctx_reset): Use bcrypt_ctx_reset()

Modified:
   subversion/trunk/subversion/libsvn_subr/checksum_bcrypt.c

Modified: subversion/trunk/subversion/libsvn_subr/checksum_bcrypt.c
==============================================================================
--- subversion/trunk/subversion/libsvn_subr/checksum_bcrypt.c	Sat Jan  3 23:20:04 2026	(r1931080)
+++ subversion/trunk/subversion/libsvn_subr/checksum_bcrypt.c	Sat Jan  3 23:37:04 2026	(r1931081)
@@ -152,6 +152,16 @@ bcrypt_ctx_final(algorithm_state_t *algo
 }
 
 static svn_error_t *
+bcrypt_ctx_reset(algorithm_state_t *algorithm,
+                 bcrypt_ctx_t *ctx)
+{
+  bcrypt_ctx_cleanup(ctx);
+  SVN_ERR(bcrypt_ctx_init(ctx, algorithm));
+
+  return SVN_NO_ERROR;
+}
+
+static svn_error_t *
 bcrypt_checksum(algorithm_state_t *algorithm,
                 unsigned char *digest,
                 const void *data,
@@ -207,10 +217,7 @@ svn_checksum__md5_ctx_create(apr_pool_t
 svn_error_t *
 svn_checksum__md5_ctx_reset(svn_checksum__md5_ctx_t *ctx)
 {
-  bcrypt_ctx_cleanup(&ctx->bcrypt_ctx);
-  SVN_ERR(bcrypt_ctx_init(&ctx->bcrypt_ctx, &md5));
-
-  return SVN_NO_ERROR;
+  return svn_error_trace(bcrypt_ctx_reset(&md5, &ctx->bcrypt_ctx));
 }
 
 svn_error_t *
@@ -261,10 +268,7 @@ svn_checksum__sha1_ctx_create(apr_pool_t
 svn_error_t *
 svn_checksum__sha1_ctx_reset(svn_checksum__sha1_ctx_t *ctx)
 {
-  bcrypt_ctx_cleanup(&ctx->bcrypt_ctx);
-  SVN_ERR(bcrypt_ctx_init(&ctx->bcrypt_ctx, &sha1));
-
-  return SVN_NO_ERROR;
+  return svn_error_trace(bcrypt_ctx_reset(&sha1, &ctx->bcrypt_ctx));
 }
 
 svn_error_t *
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.