From bd046764e31267c83e7ae515d9bc7f09ffaa5b95 Mon Sep 17 00:00:00 2001
From: Nobuyoshi Nakada <nobu@ruby-lang.org>
Date: Wed, 29 Mar 2023 18:42:46 +0900
Subject: [PATCH] [Bug #19549] Check for variables to be interpolated

---
 parse.y                 | 18 ++----------------
 test/ruby/test_parse.rb |  5 +++++
 2 files changed, 7 insertions(+), 16 deletions(-)

diff --git a/parse.y b/parse.y
index 8b88b1b38df6a8..277d313592ce9d 100644
--- a/parse.y
+++ b/parse.y
@@ -5432,24 +5432,10 @@ string_dend	: tSTRING_DEND
                 | END_OF_INPUT
                 ;
 
-string_dvar	: tGVAR
+string_dvar	: nonlocal_var
                     {
                     /*%%%*/
-                        $$ = NEW_GVAR($1, &@$);
-                    /*% %*/
-                    /*% ripper: var_ref!($1) %*/
-                    }
-                | tIVAR
-                    {
-                    /*%%%*/
-                        $$ = NEW_IVAR($1, &@$);
-                    /*% %*/
-                    /*% ripper: var_ref!($1) %*/
-                    }
-                | tCVAR
-                    {
-                    /*%%%*/
-                        $$ = NEW_CVAR($1, &@$);
+                        if (!($$ = gettable(p, $1, &@$))) $$ = NEW_BEGIN(0, &@$);
                     /*% %*/
                     /*% ripper: var_ref!($1) %*/
                     }
diff --git a/test/ruby/test_parse.rb b/test/ruby/test_parse.rb
index cf989d190b20f0..957a37eb81bb8a 100644
--- a/test/ruby/test_parse.rb
+++ b/test/ruby/test_parse.rb
@@ -1443,6 +1443,11 @@ def set(arg)
     assert_equal(expected, obj.arg)
   end
 
+  def test_ungettable_gvar
+    assert_syntax_error('$01234', /not valid to get/)
+    assert_syntax_error('"#$01234"', /not valid to get/)
+  end
+
 =begin
   def test_past_scope_variable
     assert_warning(/past scope/) {catch {|tag| eval("BEGIN{throw tag}; tap {a = 1}; a")}}