devdanzin/hamt_bitmap_leak.md

Created March 18, 2026 10:06

Star (0) You must be signed in to star a gist
Fork (0) You must be signed in to fork a gist

Select an option

Learn more about clone URLs
Clone this repository at <script src="https://gist.github.com/devdanzin/332a120e4c7750e8a7aff47419311696.js"></script>
Save devdanzin/332a120e4c7750e8a7aff47419311696 to your computer and use it in GitHub Desktop.

Download ZIP

hamt.c: sub_node leaked in bitmap_assoc and bitmap_without

Raw

hamt_bitmap_leak.md

hamt.c: sub_node leaked in bitmap_assoc and bitmap_without

Summary

Two analogous bugs:

hamt_node_bitmap_assoc (line 704): Recursive hamt_node_assoc returns new sub_node, then hamt_node_bitmap_clone fails → sub_node leaked. Line 755 correctly DECREFs.
hamt_node_bitmap_without (line 997): Same pattern. Line 968 correctly DECREFs.

Fix

Add Py_DECREF(sub_node) before return NULL/return W_ERROR at lines 705 and 997.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment