Skip to content

Commit 6b985a3

Browse files
DinoVmeta-codesync[bot]
authored andcommitted
Fix potential memory leak in TypedArgument
Summary: Another Claude identified leak. We're using `pytype.release();` instead of `pytype.reset();` and then he also decided to fix up the assignment operator. Reviewed By: jbower-fb Differential Revision: D91513285 fbshipit-source-id: 2c45c53f3cd74bf0f3614bd5eac724a833a56704
1 parent fc0c172 commit 6b985a3

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

cinderx/Jit/hir/hir.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1197,7 +1197,7 @@ TypedArgument::TypedArgument(
11971197

11981198
TypedArgument::~TypedArgument() {
11991199
ThreadedCompileSerialize guard;
1200-
pytype.release();
1200+
pytype.reset();
12011201
}
12021202

12031203
TypedArgument::TypedArgument(const TypedArgument& other)

0 commit comments

Comments
 (0)