summaryloggraphiclog1graphiclog2commitcommitdifftreerefseditfork
rawpatchinlineside by side (parent: 70ba310)
rawpatchinlineside by side (parent: 70ba310)
| author | Peter Zhu <[email protected]> | |
| Wed, 15 Mar 2023 03:18:11 +0000 (14 23:18 -0400) | ||
| committer | Peter Zhu <[email protected]> | |
| Wed, 15 Mar 2023 03:18:11 +0000 (14 23:18 -0400) |
[Bug #19529]
The fix for [Bug #19529] in commit 548086b contained a bug that crashes
on the following script:
```
wm = ObjectSpace::WeakMap.new
obj = Object.new
100.times do
wm[Object.new] = obj
GC.start
end
GC.compact
```
The fix for [Bug #19529] in commit 548086b contained a bug that crashes
on the following script:
```
wm = ObjectSpace::WeakMap.new
obj = Object.new
100.times do
wm[Object.new] = obj
GC.start
end
GC.compact
```
| test/ruby/test_weakmap.rb | patchblobblamehistory | |
| weakmap.c | patchblobblamehistory |
GC.compact
@wm[i] = obj
end
+
+ assert_separately(%w(--disable-gems), <<-'end;')
+ wm = ObjectSpace::WeakMap.new
+ obj = Object.new
+ 100.times do
+ wm[Object.new] = obj
+ GC.start
+ end
+ GC.compact
+ end;
end
end
diff --git a/weakmap.c b/weakmap.c
--- a/weakmap.c
+++ b/weakmap.c
{
*key = rb_gc_location((VALUE)*key);
- VALUE *values = (VALUE *)value;
+ VALUE *values = (VALUE *)*value;
VALUE size = values[0];
for (VALUE index = 1; index <= size; index++) {