Skip to content

Commit 48bb722

Browse files
committed
extract_utils: use copy.deepcopy() for merging fixups
Previously it'd end up merging unrelated fixups since they'd share the same addresses, yikes! Change-Id: Iedceeef3ea66436e79cda9f5f86f4dd5a2af3101
1 parent 9b49cf2 commit 48bb722

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

extract_utils/fixups.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55

66
from __future__ import annotations
77

8+
import copy
89
from typing import (
910
Dict,
1011
Optional,
@@ -32,13 +33,13 @@ def flatten_fixups(
3233
if entries in fixups_final:
3334
fixups_final[entries].merge(value)
3435
else:
35-
fixups_final[entries] = value
36+
fixups_final[entries] = copy.deepcopy(value)
3637
else:
3738
assert isinstance(entries, tuple)
3839
for entry in entries:
3940
if entry in fixups_final:
4041
fixups_final[entry].merge(value)
4142
else:
42-
fixups_final[entry] = value
43+
fixups_final[entry] = copy.deepcopy(value)
4344

4445
return fixups_final

0 commit comments

Comments
 (0)