Merge pull request #556 from ionut-arm/fix-clippy

Fix clippy warnings
parallaxsecond · Dec 3, 2024 · acec889 · acec889
2 parents 2dd2308 + c4fc6db
commit acec889
Show file tree

Hide file tree

Showing 8 changed files with 107 additions and 107 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -88,6 +88,7 @@ jobs:
       - name: Run the tests
         run: docker run -v $(pwd):/tmp/rust-tss-esapi -w /tmp/rust-tss-esapi/tss-esapi --security-opt seccomp=unconfined fedoracontainer dbus-run-session -- /tmp/rust-tss-esapi/tss-esapi/tests/all-fedora.sh
 
+  # SEE #558 : remove use of 1.82 once 1.84 rolls out;
   tests-valgrind:
     name: Valgrind test run
     runs-on: ubuntu-latest
@@ -96,7 +97,7 @@ jobs:
       - name: Build the container
         run: docker build -t ubuntucontainer tss-esapi/tests/ --file tss-esapi/tests/Dockerfile-ubuntu --target tpm2-tools
       - name: Run the tests
-        run: docker run -v $(pwd):/tmp/rust-tss-esapi -w /tmp/rust-tss-esapi/tss-esapi ubuntucontainer /tmp/rust-tss-esapi/tss-esapi/tests/valgrind.sh
+        run: docker run -v $(pwd):/tmp/rust-tss-esapi -w /tmp/rust-tss-esapi/tss-esapi -e RUST_TOOLCHAIN_VERSION="1.82" ubuntucontainer /tmp/rust-tss-esapi/tss-esapi/tests/valgrind.sh
 
   # Check that the documentation builds as well.
   docs:

diff --git a/tss-esapi/src/constants/mod.rs b/tss-esapi/src/constants/mod.rs
@@ -1,8 +1,8 @@
 // Copyright 2020 Contributors to the Parsec project.
 // SPDX-License-Identifier: Apache-2.0
 
-/// This module contains both the constants defined in the TSS specification (tss module)
-/// but also the internal representation of the TSS constants.
+//! This module contains both the constants defined in the TSS specification (tss module)
+//! but also the internal representation of the TSS constants.
 
 /// Representation of the constants defined in the
 /// Constants -> TPM_ALG_ID section of the specification

diff --git a/tss-esapi/src/ffi.rs b/tss-esapi/src/ffi.rs
@@ -38,7 +38,7 @@ where
 /// The owned bytes in the form of a `Vec<u8>` object.
 pub fn to_owned_bytes(ffi_bytes_ptr: *mut u8, size: usize) -> Vec<u8> {
     let ffi_bytes = unsafe { Malloced::<[u8]>::slice_from_raw_parts(ffi_bytes_ptr, size) };
-    return Vec::<u8>::from(ffi_bytes.as_ref());
+    Vec::<u8>::from(ffi_bytes.as_ref())
 }
 
 /// Type used for handling `size_t` variables

diff --git a/tss-esapi/src/handles/mod.rs b/tss-esapi/src/handles/mod.rs
@@ -4,10 +4,9 @@
 /// Module that contains the different types of handles
 /// that the ESAPI and the TPM uses in order to provide
 /// access to objects that was or has been created.
-
-/////////////////////////////////////////////////////////
-/// ESYS Handles
-/////////////////////////////////////////////////////////
+// ///////////////////////////////////////////////////////
+// ESYS Handles
+// //////////////////////////////////////////////////////
 pub use handle::auth::AuthHandle;
 pub use handle::key::KeyHandle;
 pub use handle::nv_index::NvIndexHandle;

diff --git a/tss-esapi/src/interface_types/reserved_handles.rs b/tss-esapi/src/interface_types/reserved_handles.rs
@@ -8,11 +8,9 @@ use crate::{
     Error, Result, WrapperErrorKind,
 };
 use std::convert::TryFrom;
-//////////////////////////////////////////////////////////////////////////////////
 /// Hierarchy
 ///
 /// Enum describing the object hierarchies in a TPM 2.0.
-//////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash)]
 pub enum Hierarchy {
     Owner,
@@ -74,9 +72,9 @@ impl TryFrom<TpmHandle> for Hierarchy {
     }
 }
 
-//////////////////////////////////////////////////////////////////////////////////
-/// Enables
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Enables
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Enables {
     Owner,
@@ -143,9 +141,9 @@ impl TryFrom<TpmHandle> for Enables {
     }
 }
 
-//////////////////////////////////////////////////////////////////////////////////
-/// HierarchyAuth
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// HierarchyAuth
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum HierarchyAuth {
     Owner,
@@ -206,13 +204,13 @@ impl TryFrom<TpmHandle> for HierarchyAuth {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// TODO: HierarchyPolicy
-//////////////////////////////////////////////////////////////////////////////////
+// ///////////////////////////////////////////////////////////////////////////////
+// TODO: HierarchyPolicy
+// ///////////////////////////////////////////////////////////////////////////////
 
-//////////////////////////////////////////////////////////////////////////////////
-/// Platform
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Platform
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Platform {
     Platform,
@@ -234,9 +232,9 @@ impl TryFrom<AuthHandle> for Platform {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// Owner
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Owner
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Owner {
     Owner,
@@ -263,9 +261,9 @@ impl TryFrom<ObjectHandle> for Owner {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// Endorsement
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Endorsement
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Endorsement {
     Endorsement,
@@ -292,9 +290,9 @@ impl TryFrom<ObjectHandle> for Endorsement {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// Provision
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Provision
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Provision {
     Owner,
@@ -321,9 +319,9 @@ impl TryFrom<AuthHandle> for Provision {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// Clear
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Clear
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Clear {
     Owner,
@@ -350,9 +348,9 @@ impl TryFrom<AuthHandle> for Clear {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// NvAuth
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// NvAuth
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum NvAuth {
     Platform,
@@ -381,9 +379,9 @@ impl TryFrom<AuthHandle> for NvAuth {
         }
     }
 }
-//////////////////////////////////////////////////////////////////////////////////
-/// Lockout
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// Lockout
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum Lockout {
     Lockout,
@@ -406,9 +404,9 @@ impl TryFrom<ObjectHandle> for Lockout {
     }
 }
 
-//////////////////////////////////////////////////////////////////////////////////
-/// NvIndex
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// NvIndex
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum NvIndex {
     NvIndex(NvIndexTpmHandle),
@@ -428,9 +426,9 @@ impl From<NvIndex> for NvIndexTpmHandle {
     }
 }
 
-//////////////////////////////////////////////////////////////////////////////////
-/// AttachedComponent
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
+// AttachedComponent
+// ////////////////////////////////////////////////////////////////////////////////
 #[derive(Debug, Clone, Copy, PartialEq, Eq)]
 pub enum AttachedComponent {
     AttachedComponent(AttachedComponentTpmHandle),
@@ -450,8 +448,8 @@ impl From<AttachedComponent> for AttachedComponentTpmHandle {
     }
 }
 
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////
 // Act (authenticated timers)
 //
 // TODO: Figure out how to implement this. This is some kind of counter.
-//////////////////////////////////////////////////////////////////////////////////
+// ////////////////////////////////////////////////////////////////////////////////