From d389bb562d1d874a84ec55d131374d02448d1be8 Mon Sep 17 00:00:00 2001 From: Cameron McCormack Date: Tue, 14 Apr 2020 17:47:25 +0000 Subject: [PATCH] Bug 1629779 - Avoid cert_storage rkv option warning. r=keeler Differential Revision: https://phabricator.services.mozilla.com/D70805 --HG-- extra : moz-landing-system : lando --- security/manager/ssl/cert_storage/src/lib.rs | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/security/manager/ssl/cert_storage/src/lib.rs b/security/manager/ssl/cert_storage/src/lib.rs index 9d0874d8b5e5..ff116d7d7b1c 100644 --- a/security/manager/ssl/cert_storage/src/lib.rs +++ b/security/manager/ssl/cert_storage/src/lib.rs @@ -878,7 +878,14 @@ fn get_store_path(profile_path: &PathBuf) -> Result fn make_env(path: &Path) -> Result { let mut builder = Rkv::environment_builder::(); builder.set_max_dbs(2); - builder.set_map_size(16777216); // 16MB + + // 16MB is a little over twice the size of the current dataset. When we + // eventually switch to the LMDB backend to create the builder above, + // we should set this as the map size, since it cannot currently resize. + // (The SafeMode backend warns when a map size is specified, so we skip it + // for now to avoid console spam.) + + // builder.set_map_size(16777216); // Bug 1595004: Migrate databases between backends in the future, // and handle 32 and 64 bit architectures in case of LMDB.