From 0cf7fb26d30a42b48132a25f325fb0b33790a2de Mon Sep 17 00:00:00 2001
From: Bilal Elmoussaoui <belmouss@redhat.com>
Date: Wed, 4 Dec 2024 14:34:33 +0100
Subject: [PATCH] server: Remove unecessary clone

Session is already Clone, no need for an extra Arc wrapper.
---
 server/src/service.rs | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

diff --git a/server/src/service.rs b/server/src/service.rs
index 69d59239..515b9997 100644
--- a/server/src/service.rs
+++ b/server/src/service.rs
@@ -27,7 +27,7 @@ pub struct Service {
     // Other attributes
     connection: zbus::Connection,
     // sessions mapped to their corresponding object path on the bus
-    sessions: Arc<Mutex<HashMap<OwnedObjectPath, Arc<Session>>>>,
+    sessions: Arc<Mutex<HashMap<OwnedObjectPath, Session>>>,
     session_index: Arc<RwLock<u32>>,
 }
 
@@ -55,13 +55,10 @@ impl Service {
         let session = Session::new(aes_key.map(Arc::new), self.clone()).await;
         let path = session.path().clone();
 
-        {
-            let session = Arc::new(session.clone());
-            self.sessions
-                .lock()
-                .await
-                .insert(path.clone(), Arc::clone(&session));
-        }
+        self.sessions
+            .lock()
+            .await
+            .insert(path.clone(), session.clone());
 
         object_server.at(&path, session).await?;