From 7582287eb27e6b649789ce31ffdcbbb9ffcaf726 Mon Sep 17 00:00:00 2001
From: Jeremy Evans <code@jeremyevans.net>
Date: Tue, 18 Jun 2019 18:59:49 -0700
Subject: Make String#-@ not freeze receiver if called on unfrozen subclass
 instance

rb_fstring behavior in this case is to freeze the receiver.  I'm
not sure if that should be changed, so this takes the conservative
approach of duping the receiver in String#-@ before passing
to rb_fstring.

Fixes [Bug #15926]
---
 test/ruby/test_string.rb | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

(limited to 'test/ruby')

diff --git a/test/ruby/test_string.rb b/test/ruby/test_string.rb
index 507e067a0d..26de52dedb 100644
--- a/test/ruby/test_string.rb
+++ b/test/ruby/test_string.rb
@@ -3175,6 +3175,22 @@ CODE
     assert_same(str, -bar, "uminus deduplicates [Feature #13077]")
   end
 
+  def test_uminus_no_freeze_not_bare
+    str = @cls.new("foo")
+    -str
+    assert_equal(false, str.frozen?)
+
+    str = @cls.new("foo")
+    str.instance_variable_set(:@iv, 1)
+    -str
+    assert_equal(false, str.frozen?)
+
+    str = @cls.new("foo")
+    str.taint
+    -str
+    assert_equal(false, str.frozen?)
+  end
+
   def test_ord
     assert_equal(97, "a".ord)
     assert_equal(97, "abc".ord)
-- 
cgit v1.2.3