remove unnecessary std::min call

max_file_size_rejected is a u32 always less than first argument (1ull << 44)
wallix · Mar 26, 2024 · 1de03bc · 1de03bc
1 parent 1be26b5
commit 1de03bc
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/acl/module_manager/create_module_rdp.cpp b/src/acl/module_manager/create_module_rdp.cpp
@@ -746,10 +746,9 @@ ModPack create_mod_rdp(
     vp.log_if_accepted = ini.get<cfg::file_verification::log_if_accepted>();
     vp.block_invalid_file_up = ini.get<cfg::file_verification::block_invalid_file_up>();
     vp.block_invalid_file_down = ini.get<cfg::file_verification::block_invalid_file_down>();
-    vp.max_file_size_rejected = std::min<uint64_t>(
-        (1ull << (64 - 20 /* mebibyte */)) - 1u,
-        ini.get<cfg::file_verification::max_file_size_rejected>()
-    ) * 1024u * 1024u;
+    vp.max_file_size_rejected
+      = safe_cast<uint64_t>(ini.get<cfg::file_verification::max_file_size_rejected>())
+      * 1024u * 1024u /* mebibyte to byte */;
     vp.enable_clipboard_text_up = ini.get<cfg::file_verification::clipboard_text_up>();
     vp.enable_clipboard_text_down = ini.get<cfg::file_verification::clipboard_text_down>();
     vp.block_invalid_text_up = ini.get<cfg::file_verification::block_invalid_clipboard_text_up>();