[GR-70811] Fix PE in TextIOWrapperNodes.CheckClosedNode

andrykonchin · andrykonchin · commit a682556e72e2 · 2025-10-16T17:49:06.000Z
PullRequest: graalpython/4059
diff --git a/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/io/TextIOWrapperNodes.java b/graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/io/TextIOWrapperNodes.java
@@ -183,11 +183,10 @@ static void error(@SuppressWarnings("unused") PTextIO self,
         static void checkGeneric(VirtualFrame frame, PTextIO self,
                         @Bind Node inliningTarget,
                         @Cached PyObjectGetAttr getAttr,
-                        @Cached PyObjectIsTrueNode isTrueNode,
-                        @Cached PRaiseNode raiseNode) {
+                        @Cached PyObjectIsTrueNode isTrueNode) {
             Object res = getAttr.execute(frame, inliningTarget, self.getBuffer(), T_CLOSED);
             if (isTrueNode.execute(frame, res)) {
-                error(self, raiseNode);
+                error(self, inliningTarget);
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -183,11 +183,10 @@ static void error(@SuppressWarnings("unused") PTextIO self,`
`183`	`183`	`static void checkGeneric(VirtualFrame frame, PTextIO self,`
`184`	`184`	`@Bind Node inliningTarget,`
`185`	`185`	`@Cached PyObjectGetAttr getAttr,`
`186`		`- @Cached PyObjectIsTrueNode isTrueNode,`
`187`		`- @Cached PRaiseNode raiseNode) {`
	`186`	`+ @Cached PyObjectIsTrueNode isTrueNode) {`
`188`	`187`	`Object res = getAttr.execute(frame, inliningTarget, self.getBuffer(), T_CLOSED);`
`189`	`188`	`if (isTrueNode.execute(frame, res)) {`
`190`		`- error(self, raiseNode);`
	`189`	`+ error(self, inliningTarget);`
`191`	`190`	`}`
`192`	`191`	`}`
`193`	`192`	`}`