helium / blockchain-core

Apache License 2.0
213 stars 86 forks source link

Handle `undefined` `poc_witness_consideration_limit` #1297

Closed xandkar closed 2 years ago

xandkar commented 2 years ago

Looks like it is set to undefined by blockchain_utils:get_vars during blockchain_txn_poc_receipts_v1:check_is_valid_poc, but blockchain_poc_target_v3:limit_addrs assumes that a present value in the map is an integer, which later causes a crash in lists:sublist within blockchain_utils:deterministic_subset, which tries to use undefined as a sublist length.