Project

General

Profile

Bug #9645

[PATCH] variable.c: avoid memory leak on const redefinition

Added by Eric Wong over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
ruby -v:
ruby 2.2.0dev (2014-03-16 trunk 45349) [x86_64-linux]
[ruby-core:61546]

Description

I noticed this leak because I was making constants use ihash from Feature 9614.
I hope nobody is really affected by this leak.

Associated revisions

Revision 45350
Added by normal over 2 years ago

variable.c: avoid memory leak on const redefinition

  • variable.c (rb_const_set): delete existing entry on redefinition [Bug #9645]
  • test/ruby/test_const.rb (test_redefinition): test for leak

Revision 45350
Added by normal over 2 years ago

variable.c: avoid memory leak on const redefinition

  • variable.c (rb_const_set): delete existing entry on redefinition [Bug #9645]
  • test/ruby/test_const.rb (test_redefinition): test for leak

Revision 45350
Added by normal over 2 years ago

variable.c: avoid memory leak on const redefinition

  • variable.c (rb_const_set): delete existing entry on redefinition [Bug #9645]
  • test/ruby/test_const.rb (test_redefinition): test for leak

Revision 45373
Added by Tomoyuki Chikanaga over 2 years ago

merge revision(s) r45350: [Backport #9645]

* variable.c (rb_const_set): delete existing entry on redefinition
  [Bug #9645]

* test/ruby/test_const.rb (test_redefinition): test for leak

Revision 46301
Added by Tomoyuki Chikanaga over 2 years ago

merge revision(s) r45350: [Backport #9645]

* variable.c (rb_const_set): delete existing entry on redefinition
  [Bug #9645]

* test/ruby/test_const.rb (test_redefinition): test for leak

History

#1 Updated by Anonymous over 2 years ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

Applied in changeset r45350.


variable.c: avoid memory leak on const redefinition

  • variable.c (rb_const_set): delete existing entry on redefinition [Bug #9645]
  • test/ruby/test_const.rb (test_redefinition): test for leak

#2 [ruby-core:61610] Updated by Tomoyuki Chikanaga over 2 years ago

  • Backport changed from 2.0.0: REQUIRED, 2.1: REQUIRED to 2.0.0: DONE, 2.1: REQUIRED

r45350 was backported to ruby_2_0_0 at r45373.

#3 [ruby-core:62886] Updated by Tomoyuki Chikanaga over 2 years ago

  • Backport changed from 2.0.0: DONE, 2.1: REQUIRED to 2.0.0: DONE, 2.1: DONE

Backported into ruby_2_1 branch at r46301.

Also available in: Atom PDF