Rebase onto main

solana-labs · Oct 21, 2024 · 32f43bb · 32f43bb
1 parent 2f44c65
commit 32f43bb
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/src/elf.rs b/src/elf.rs
@@ -1072,7 +1072,7 @@ impl<C: ContextObject> Executable<C> {
                             .or_insert_with(|| ebpf::hash_symbol_name(name));
                         if config.reject_broken_elfs
                             && loader
-                                .get_function_registry(&SBPFVersion::V1)
+                                .get_function_registry(SBPFVersion::V1)
                                 .lookup_by_key(hash)
                                 .is_none()
                         {

diff --git a/src/program.rs b/src/program.rs
@@ -154,7 +154,7 @@ impl<T: Copy + PartialEq> FunctionRegistry<T> {
                 ebpf::hash_symbol_name(&usize::from(value).to_le_bytes())
             };
             if loader
-                .get_function_registry(&SBPFVersion::V1)
+                .get_function_registry(SBPFVersion::V1)
                 .lookup_by_key(hash)
                 .is_some()
             {
@@ -292,9 +292,9 @@ impl<C: ContextObject> BuiltinProgram<C> {
     /// Get the function registry depending on the SBPF version
     pub fn get_function_registry(
         &self,
-        sbpf_version: &SBPFVersion,
+        sbpf_version: SBPFVersion,
     ) -> &FunctionRegistry<BuiltinFunction<C>> {
-        if sbpf_version == &SBPFVersion::V1 {
+        if sbpf_version == SBPFVersion::V1 {
             &self.sparse_registry
         } else {
             &self.dense_registry

diff --git a/src/static_analysis.rs b/src/static_analysis.rs
@@ -224,11 +224,7 @@ impl<'a> Analysis<'a> {
     /// Splits the sequence of instructions into basic blocks
     ///
     /// Also links the control-flow graph edges between the basic blocks.
-    pub fn split_into_basic_blocks(
-        &mut self,
-        flatten_call_graph: bool,
-        sbpf_version: &SBPFVersion,
-    ) {
+    pub fn split_into_basic_blocks(&mut self, flatten_call_graph: bool, sbpf_version: SBPFVersion) {
         self.cfg_nodes.insert(0, CfgNode::default());
         for pc in self.functions.keys() {
             self.cfg_nodes.entry(*pc).or_default();