cleanup of allocate_accounts_index (#19585)

solana-labs · Sep 2, 2021 · 682daf1 · 682daf1
1 parent 78585a9
commit 682daf1
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/runtime/src/accounts_index.rs b/runtime/src/accounts_index.rs
@@ -830,11 +830,12 @@ impl<T: IsCached> AccountsIndex<T> {
         let bins = config
             .and_then(|config| config.bins)
             .unwrap_or(BINS_DEFAULT);
+        // create bin_calculator early to verify # bins is reasonable
+        let bin_calculator = PubkeyBinCalculator16::new(bins);
         let account_maps = (0..bins)
             .into_iter()
-            .map(|_| RwLock::new(AccountMap::default()))
+            .map(|_bin| RwLock::new(AccountMap::new()))
             .collect::<Vec<_>>();
-        let bin_calculator = PubkeyBinCalculator16::new(bins);
         (account_maps, bin_calculator)
     }