Ensure all frame names are non-null

These names will leak out in stack traces so may need to be reconsidered.
jruby · Apr 5, 2024 · 18e8a0f · 18e8a0f
1 parent 823dce1
commit 18e8a0f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 4 deletions.
diff --git a/core/src/main/java/org/jruby/ir/interpreter/Interpreter.java b/core/src/main/java/org/jruby/ir/interpreter/Interpreter.java
@@ -127,15 +127,15 @@ public static IRubyObject INTERPRET_BLOCK(ThreadContext context, Block block, IR
     }
 
     public static IRubyObject INTERPRET_CLASS(ThreadContext context, IRScope body, RubyModule clazz) {
-        return interpretFrameScope(context, null, body, clazz, null, Visibility.PUBLIC, clazz, null, null, Block.NULL_BLOCK);
+        return interpretFrameScope(context, null, body, clazz, null, Visibility.PUBLIC, clazz, "<class>", null, Block.NULL_BLOCK);
     }
 
     public static IRubyObject INTERPRET_MODULE(ThreadContext context, IRScope body, RubyModule clazz) {
-        return interpretFrameScope(context, null, body, clazz, null, Visibility.PUBLIC, clazz, null, null, Block.NULL_BLOCK);
+        return interpretFrameScope(context, null, body, clazz, null, Visibility.PUBLIC, clazz, "<module>", null, Block.NULL_BLOCK);
     }
 
     public static IRubyObject INTERPRET_METACLASS(ThreadContext context, IRScope body, RubyModule clazz, Visibility visibility) {
-        return interpretFrameScope(context, null, body, clazz, context.getCurrentScope(), visibility, clazz, null, null, Block.NULL_BLOCK);
+        return interpretFrameScope(context, null, body, clazz, context.getCurrentScope(), visibility, clazz, "<metaclass>", null, Block.NULL_BLOCK);
     }
 
     public static IRubyObject INTERPRET_METHOD(ThreadContext context, IRScope body, RubyModule implClass,

diff --git a/core/src/main/java/org/jruby/runtime/Frame.java b/core/src/main/java/org/jruby/runtime/Frame.java
@@ -96,6 +96,7 @@ public final class Frame {
      * when needed.
      */
     public Frame() {
+        name = "<root>";
     }
 
     /**
@@ -112,6 +113,7 @@ private Frame(Block nullBlock) {
      */
     private Frame(Frame frame) {
         assert frame.block != null;
+        assert frame.name != null;
 
         this.self = frame.self;
         this.name = frame.name;
@@ -135,6 +137,7 @@ public void updateFrame() {
      * @param name The name of the method being called
      */
     public void updateFrame(String name) {
+        assert name != null;
         this.name = name;
     }
 
@@ -147,6 +150,7 @@ public void updateFrame(String name) {
     public void updateFrame(Frame frame) {
         Block block = frame.block;
         assert block != null;
+        assert frame.name != null;
 
         this.self = frame.self;
         this.name = frame.name;
@@ -165,6 +169,7 @@ public void updateFrame(Frame frame) {
      */
     public void updateFrame(RubyModule klazz, IRubyObject self, String name, Block block) {
         assert block != null;
+        assert name != null;
         this.self = self;
         this.name = name;
         this.klazz = klazz;
@@ -182,6 +187,7 @@ public void updateFrame(RubyModule klazz, IRubyObject self, String name, Block b
      */
     public void updateFrame(RubyModule klazz, IRubyObject self, String name, Visibility visibility, Block block) {
         assert block != null;
+        assert name != null;
         this.self = self;
         this.name = name;
         this.klazz = klazz;
@@ -196,7 +202,7 @@ public void updateFrame(RubyModule klazz, IRubyObject self, String name, Visibil
      */
     public void updateFrameForEval(IRubyObject self) {
         this.self = self;
-        this.name = null;
+        this.name = "<eval>";
         this.visibility = Visibility.PRIVATE;
     }
 
@@ -267,6 +273,7 @@ public void setKlazz(RubyModule klazz) {
      * @param name the new name
      */
     public void setName(String name) {
+        assert name != null;
         this.name = name;
     }