From cf85546863eab509552b3bad8ee60a47ca1d4bbf Mon Sep 17 00:00:00 2001 From: Brian Smith Date: Fri, 21 Oct 2022 09:34:31 -0700 Subject: [PATCH] Tests: Use custom tests to verify operations on empty slices are no-ops. Modify the custom tests so that they would have detected and prevented the issue fixed in https://github.com/rust-random/getrandom/pull/298. --- tests/custom.rs | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/custom.rs b/tests/custom.rs index 4f9cb807..b085094b 100644 --- a/tests/custom.rs +++ b/tests/custom.rs @@ -16,6 +16,9 @@ fn len7_err() -> Error { } fn super_insecure_rng(buf: &mut [u8]) -> Result<(), Error> { + // `getrandom` guarantees it will not call any implementation if the output + // buffer is empty. + assert!(!buf.is_empty()); // Length 7 buffers return a custom error if buf.len() == 7 { return Err(len7_err());