Skip to content

Commit 51178d1

Browse files
authored
Merge pull request #209 from LedgerHQ/y333_241031/heap_size_assert
Update assert on heap size
2 parents dcab944 + 6ebdd16 commit 51178d1

File tree

4 files changed

+9
-10
lines changed

4 files changed

+9
-10
lines changed

Cargo.lock

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

ledger_device_sdk/Cargo.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "ledger_device_sdk"
3-
version = "1.18.1"
3+
version = "1.18.2"
44
authors = ["yhql", "yogh333", "agrojean-ledger", "kingofpayne"]
55
edition = "2021"
66
license.workspace = true
@@ -21,7 +21,7 @@ rand_core = { version = "0.6.3", default_features = false }
2121
zeroize = { version = "1.6.0", default_features = false }
2222
numtoa = "0.2.4"
2323
const-zero = "0.1.1"
24-
ledger_secure_sdk_sys = { path = "../ledger_secure_sdk_sys", version = "1.5.1" }
24+
ledger_secure_sdk_sys = { path = "../ledger_secure_sdk_sys", version = "1.5.2" }
2525

2626
[features]
2727
speculos = []

ledger_secure_sdk_sys/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
[package]
22
name = "ledger_secure_sdk_sys"
3-
version = "1.5.1"
3+
version = "1.5.2"
44
authors = ["yhql", "agrojean-ledger"]
55
edition = "2021"
66
license.workspace = true

ledger_secure_sdk_sys/build.rs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -600,12 +600,11 @@ impl SDKBuilder {
600600
// Read the HEAP_SIZE environment variable, default to 8192 if not set
601601
let heap_size = env::var("HEAP_SIZE").unwrap_or_else(|_| "8192".to_string());
602602

603+
let heap_size_value = heap_size.parse::<u32>().unwrap();
604+
603605
assert!(
604-
match heap_size.as_str() {
605-
"2048" | "4096" | "8192" | "16384" | "24576" => true,
606-
_ => false,
607-
},
608-
"Invalid heap size: {heap_size}; Authorized sizes are 2048, 4096, 8192, 16384, 24576"
606+
heap_size_value >= 2048 && heap_size_value <= 24576,
607+
"Invalid heap size: {heap_size}; Shall be included in [2048, 24576]"
609608
);
610609

611610
// Generate the heap_size.rs file with the HEAP_SIZE value

0 commit comments

Comments
 (0)