authorgravatar for hello@nektro.netMeghan Denny <hello@nektro.net> 2026-06-09 17:01:54-07:00
committergravatar for andrew@ziglang.orgAndrew Kelley <andrew@ziglang.org> 2026-06-11 01:36:18+02:00
logfce6dbb8907c1bfc8da381632276cc3df8f783e3
treeb2d68a154276b5559218ccd4bbb69799ef103b9f
parente357134f0f02cbd458b59eaf8de3efa0cf473b35

std.crypto: argon2/scrypt: cleanup VerifyOptions

2 files changed, 6 insertions(+), 12 deletions(-)

lib/std/crypto/argon2.zig+2-5
......@@ -609,10 +609,8 @@ pub fn strHash(
609609}
610610
611611/// Options for hash verification.
612///
613/// Allocator is required for argon2.
614612pub const VerifyOptions = struct {
615 allocator: ?mem.Allocator,
613 allocator: mem.Allocator,
616614};
617615
618616/// Verify that a previously computed hash is valid for a given password.
......@@ -622,8 +620,7 @@ pub fn strVerify(
622620 options: VerifyOptions,
623621 io: Io,
624622) Error!void {
625 const allocator = options.allocator orelse return Error.AllocatorRequired;
626 return PhcFormatHasher.verify(allocator, str, password, io);
623 return PhcFormatHasher.verify(options.allocator, str, password, io);
627624}
628625
629626test "argon2d" {
lib/std/crypto/scrypt.zig+4-7
......@@ -566,10 +566,8 @@ pub fn strHashWithSalt(
566566}
567567
568568/// Options for hash verification.
569///
570/// Allocator is required for scrypt.
571569pub const VerifyOptions = struct {
572 allocator: ?mem.Allocator,
570 allocator: mem.Allocator,
573571};
574572
575573/// Verify that a previously computed hash is valid for a given password.
......@@ -578,11 +576,10 @@ pub fn strVerify(
578576 password: []const u8,
579577 options: VerifyOptions,
580578) Error!void {
581 const allocator = options.allocator orelse return Error.AllocatorRequired;
582579 if (mem.startsWith(u8, str, crypt_format.prefix)) {
583 return CryptFormatHasher.verify(allocator, str, password);
580 return CryptFormatHasher.verify(options.allocator, str, password);
584581 } else {
585 return PhcFormatHasher.verify(allocator, str, password);
582 return PhcFormatHasher.verify(options.allocator, str, password);
586583 }
587584}
588585
......@@ -693,7 +690,7 @@ test "strHash and strVerify" {
693690
694691 const password = "testpass";
695692 const params = Params.interactive;
696 const verify_options = VerifyOptions{ .allocator = alloc };
693 const verify_options: VerifyOptions = .{ .allocator = alloc };
697694 var buf: [128]u8 = undefined;
698695
699696 {