Skip to content

Commit 86fb4e3

Browse files
committed
MB-54516: Fix history_retention_bytes parsing
We should be parsing this value using stoull, not stoul, as longs are 32-bit on 64-bit Windows. Change-Id: Ia4302bdcc74964baaa0b9c7fbaac96469c37de79 Reviewed-on: https://review.couchbase.org/c/kv_engine/+/185029 Well-Formed: Restriction Checker Tested-by: Jim Walker <[email protected]> Reviewed-by: Jim Walker <[email protected]>
1 parent eef0b81 commit 86fb4e3

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

engines/ep/src/ep_engine.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -850,7 +850,7 @@ cb::engine_errc EventuallyPersistentEngine::setFlushParam(
850850
} else if (key == "history_retention_seconds") {
851851
configuration.setHistoryRetentionSeconds(std::stoul(val));
852852
} else if (key == "history_retention_bytes") {
853-
configuration.setHistoryRetentionBytes(std::stoul(val));
853+
configuration.setHistoryRetentionBytes(std::stoull(val));
854854
} else {
855855
msg = "Unknown config param";
856856
rv = cb::engine_errc::invalid_arguments;

0 commit comments

Comments
 (0)