From e91c62e1658c5e664bd9155132c8a80058107fbb Mon Sep 17 00:00:00 2001
From: Adam <lil22duckling@gmail.com>
Date: Sat, 15 Nov 2025 11:22:16 +0100
Subject: [PATCH] fix: optimize AccountId cloning in
 AuthorityKeys::from_accounts

Reduce unnecessary cloning by reusing stash clone for session_keys
generation. Previously controller was cloned twice; now we clone once
and reuse it, eliminating redundant memory allocation.
---
 node/src/chains/definitions.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/node/src/chains/definitions.rs b/node/src/chains/definitions.rs
index 3641b95e3..76cf86fa3 100644
--- a/node/src/chains/definitions.rs
+++ b/node/src/chains/definitions.rs
@@ -75,8 +75,8 @@ impl AuthorityKeys {
 	}
 
 	pub fn from_accounts(controller: AccountId, grandpa: GrandpaId) -> Self {
-		let session_keys = session_keys(controller.clone(), grandpa);
 		let stash = controller.clone();
+		let session_keys = session_keys(stash.clone(), grandpa);
 
 		Self {
 			controller,