GHSA-rwq6-crjg-9cpw

Source

https://github.com/advisories/GHSA-rwq6-crjg-9cpw

Import Source

https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2024/09/GHSA-rwq6-crjg-9cpw/GHSA-rwq6-crjg-9cpw.json

JSON Data

https://api.osv.dev/v1/vulns/GHSA-rwq6-crjg-9cpw

Aliases

CVE-2024-7884
RUSTSEC-2024-0372

Published

2024-09-05T16:44:27Z

Modified

2024-09-09T14:05:39Z

Severity

7.5 (High) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H CVSS Calculator
8.7 (High) CVSS_V4 - CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:N/VI:N/VA:H/SC:N/SI:N/SA:N CVSS Calculator

Summary

ic-cdk has a memory leak when calling a canister method via `ic_cdk::call`

Details

When a canister method is called via ic_cdk::call*, a new Future CallFuture is created and can be awaited by the caller to get the execution result. Internally, the state of the Future is tracked and stored in a struct called CallFutureState. A bug in the polling implementation of the CallFuture allows multiple references to be held for this internal state and not all references were dropped before the Future is resolved. Since we have unaccounted references held, a copy of the internal state ended up being persisted in the canister's heap and thus causing a memory leak.

Impact

Canisters built in Rust with ic_cdk and ic_cdk_timers are affected. If these canisters call a canister method, use timers or heartbeat, they will likely leak a small amount of memory on every such operation. In the worst case, this could lead to heap memory exhaustion triggered by an attacker.

Motoko based canisters are not affected by the bug.

Patches

The patch has been backported to all minor versions between >= 0.8.0, <= 0.15.0. The patched versions available are 0.8.2, 0.9.3, 0.10.1, 0.11.6, 0.12.2, 0.13.5, 0.14.1, 0.15.1 and their previous versions have been yanked.

Workarounds

There are no known workarounds at the moment. Developers are recommended to upgrade their canister as soon as possible to the latest available patched version of ic_cdk to avoid running out of Wasm heap memory.

[!NOTE]
Upgrading the canisters (without updating ic_cdk) also frees the leaked memory but it's only a temporary solution.

References

References

Affected packages

crates.io / ic_cdk

Package

Name: ic_cdk; View open source insights on deps.dev
Purl: pkg:cargo/ic_cdk

Affected ranges

Type: SEMVER
Events: Introduced

0.8.0

Fixed

0.8.2

crates.io / ic_cdk

Package

Name: ic_cdk; View open source insights on deps.dev
Purl: pkg:cargo/ic_cdk

Affected ranges

Type: SEMVER
Events: Introduced

0.9.0

Fixed

0.9.3

crates.io / ic_cdk

Package

Name: ic_cdk; View open source insights on deps.dev
Purl: pkg:cargo/ic_cdk

Affected ranges

Type: SEMVER
Events: Introduced

0.10.0

Fixed

0.10.1

crates.io / ic_cdk

Package

Name: ic_cdk; View open source insights on deps.dev
Purl: pkg:cargo/ic_cdk

Affected ranges

Type: SEMVER
Events: Introduced

0.11.0

Fixed

0.11.6

Database specific

{
    "last_known_affected_version_range": "< 0.11.5"
}

crates.io / ic_cdk

Package

Name: ic_cdk; View open source insights on deps.dev
Purl: pkg:cargo/ic_cdk

Affected ranges

Type: SEMVER
Events: Introduced

0.12.0

Fixed

0.12.2

crates.io / ic_cdk

Package

Name: ic_cdk; View open source insights on deps.dev
Purl: pkg:cargo/ic_cdk

Affected ranges

Type: SEMVER
Events: Introduced

0.13.0

Fixed

0.13.5

Database specific

{
    "last_known_affected_version_range": "< 0.13.4"
}