Use hybrid_array::slice_as_chunks

RustCrypto · Nov 12, 2023 · a61ac8f · a61ac8f
1 parent a92cf0d
commit a61ac8f
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 73 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/universal-hash/Cargo.lock b/universal-hash/Cargo.lock
diff --git a/universal-hash/Cargo.toml b/universal-hash/Cargo.toml
@@ -5,7 +5,7 @@ description = "Traits which describe the functionality of universal hash functio
 authors = ["RustCrypto Developers"]
 license = "MIT OR Apache-2.0"
 edition = "2021"
-rust-version = "1.56"
+rust-version = "1.65"
 readme = "README.md"
 documentation = "https://docs.rs/universal-hash"
 repository = "https://github.com/RustCrypto/traits"

diff --git a/universal-hash/README.md b/universal-hash/README.md
@@ -15,7 +15,7 @@ See [RustCrypto/universal-hashes] for implementations which use this trait.
 
 ## Minimum Supported Rust Version
 
-Rust **1.56** or higher.
+Rust **1.65** or higher.
 
 Minimum supported Rust version can be changed in the future, but it will be
 done with a minor version bump.
@@ -47,7 +47,7 @@ dual licensed as above, without any additional terms or conditions.
 [docs-image]: https://docs.rs/universal-hash/badge.svg
 [docs-link]: https://docs.rs/universal-hash/
 [license-image]: https://img.shields.io/badge/license-Apache2.0/MIT-blue.svg
-[rustc-image]: https://img.shields.io/badge/rustc-1.56+-blue.svg
+[rustc-image]: https://img.shields.io/badge/rustc-1.65+-blue.svg
 [chat-image]: https://img.shields.io/badge/zulip-join_chat-blue.svg
 [chat-link]: https://rustcrypto.zulipchat.com/#narrow/stream/260051-universal-hashes
 [build-image]: https://github.com/RustCrypto/traits/workflows/universal-hash/badge.svg?branch=master&event=push

diff --git a/universal-hash/src/lib.rs b/universal-hash/src/lib.rs
@@ -37,7 +37,7 @@ pub use crypto_common::{
 
 use core::slice;
 use crypto_common::{
-    array::{Array, ArraySize},
+    array::{slice_as_chunks, Array},
     BlockSizeUser, BlockSizes, ParBlocksSizeUser,
 };
 use subtle::ConstantTimeEq;
@@ -94,7 +94,7 @@ pub trait UniversalHash: BlockSizeUser + Sized {
             fn call<B: UhfBackend<BlockSize = BS>>(self, backend: &mut B) {
                 let pb = B::ParBlocksSize::USIZE;
                 if pb > 1 {
-                    let (par_blocks, tail) = to_blocks(self.blocks);
+                    let (par_blocks, tail) = slice_as_chunks(self.blocks);
                     for par_block in par_blocks {
                         backend.proc_par_blocks(par_block);
                     }
@@ -120,7 +120,7 @@ pub trait UniversalHash: BlockSizeUser + Sized {
     /// Message Authentication Codes (MACs) based on universal hashing.
     #[inline]
     fn update_padded(&mut self, data: &[u8]) {
-        let (blocks, tail) = to_blocks(data);
+        let (blocks, tail) = slice_as_chunks(data);
 
         self.update(blocks);
 
@@ -175,17 +175,3 @@ impl core::fmt::Display for Error {
 
 #[cfg(feature = "std")]
 impl std::error::Error for Error {}
-
-/// Split message into slice of blocks and leftover tail.
-// TODO: replace with `slice::as_chunks` on migration to const generics
-#[inline(always)]
-fn to_blocks<T, N: ArraySize>(data: &[T]) -> (&[Array<T, N>], &[T]) {
-    let nb = data.len() / N::USIZE;
-    let (left, right) = data.split_at(nb * N::USIZE);
-    let p = left.as_ptr() as *const Array<T, N>;
-    // SAFETY: we guarantee that `blocks` does not point outside of `data`
-    // and `p` is valid for reads
-    #[allow(unsafe_code)]
-    let blocks = unsafe { slice::from_raw_parts(p, nb) };
-    (blocks, right)
-}