Fix aes and kuznyechik CI workflows (#311)

Looks like they were broken in #284
RustCrypto · Mar 17, 2022 · 7fcdd7c · 7fcdd7c
1 parent 423945d
commit 7fcdd7c
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 9 deletions.
diff --git a/.github/workflows/aes.yml b/.github/workflows/aes.yml
@@ -3,6 +3,7 @@ name: aes
 on:
   pull_request:
     paths:
+      - ".github/workflows/aes.yml"
       - "aes/**"
       - "Cargo.*"
   push:
@@ -39,8 +40,8 @@ jobs:
           override: true
       - run: cargo check
       - run: |
-        cargo build --target ${{ matrix.target }}
-        cargo build --target ${{ matrix.target }} --features hazmat
+          cargo build --target ${{ matrix.target }}
+          cargo build --target ${{ matrix.target }} --features hazmat
       - env:
           RUSTFLAGS: "-Dwarnings --cfg aes_force_soft"
         run: |
@@ -205,8 +206,8 @@ jobs:
           tar -C /tmp -xzf /tmp/binaries.tar.gz
           mv /tmp/cross ~/.cargo/bin
       - run: |
-        cross test --package aes --target ${{ matrix.target }}
-        cross test --package aes --target ${{ matrix.target }} --features hazmat
+          cross test --package aes --target ${{ matrix.target }}
+          cross test --package aes --target ${{ matrix.target }} --features hazmat
       - env:
           RUSTFLAGS: "-Dwarnings --cfg aes_force_soft"
         run: |

diff --git a/.github/workflows/kuznyechik.yml b/.github/workflows/kuznyechik.yml
@@ -3,6 +3,7 @@ name: kuznyechik
 on:
   pull_request:
     paths:
+      - ".github/workflows/kuznyechik.yml"
       - "kuznyechik/**"
       - "Cargo.*"
   push:
@@ -62,8 +63,8 @@ jobs:
           override: true
           profile: minimal
       - run: |
-        cargo test
-        cargo test --all-features
+          cargo test
+          cargo test --all-features
       - env:
           RUSTFLAGS: "-Dwarnings --cfg kuznyechik_force_soft"
         run: |

diff --git a/kuznyechik/src/soft/mod.rs b/kuznyechik/src/soft/mod.rs
@@ -81,7 +81,7 @@ impl AlgorithmName for Kuznyechik {
 #[cfg_attr(docsrs, doc(cfg(feature = "zeroize")))]
 impl Drop for Kuznyechik {
     fn drop(&mut self) {
-        self.keys.zeroize();
+        self.keys.iter_mut().for_each(|key| key.zeroize());
     }
 }
 
@@ -135,7 +135,7 @@ impl AlgorithmName for KuznyechikEnc {
 #[cfg_attr(docsrs, doc(cfg(feature = "zeroize")))]
 impl Drop for KuznyechikEnc {
     fn drop(&mut self) {
-        self.keys.zeroize();
+        self.keys.iter_mut().for_each(|key| key.zeroize());
     }
 }
 
@@ -205,7 +205,7 @@ impl AlgorithmName for KuznyechikDec {
 #[cfg_attr(docsrs, doc(cfg(feature = "zeroize")))]
 impl Drop for KuznyechikDec {
     fn drop(&mut self) {
-        self.keys.zeroize();
+        self.keys.iter_mut().for_each(|key| key.zeroize());
     }
 }