jyapayne/swig
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #1177 from Sigill/sigabrt_rubyunlinkobject_fix

Browse source 
Do not abort when unlinking non-data ruby objects
This commit is contained in:
William S Fulton 2018-01-07 12:23:47 +00:00 • committed by GitHub
commit c44adff7b9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

6
Examples/ruby/free_function/runme.rb
View file

@ -39,7 +39,11 @@ GC.start
# C++ object # C++ object
ok = false ok = false
begin begin
puts tiger2.get_name # Let's stress the GC a bit, a single pass might not be enough.
10.times {
GC.start
puts tiger2.get_name
}
rescue ObjectPreviouslyDeleted => error rescue ObjectPreviouslyDeleted => error
ok = true ok = true
end end

13
Lib/ruby/rubytracking.swg
View file

@ -1,7 +1,7 @@
/* ----------------------------------------------------------------------------- /* -----------------------------------------------------------------------------
* rubytracking.swg * rubytracking.swg
* *
* This file contains support for tracking mappings from * This file contains support for tracking mappings from
* Ruby objects to C++ objects. This functionality is needed * Ruby objects to C++ objects. This functionality is needed
* to implement mark functions for Ruby's mark and sweep * to implement mark functions for Ruby's mark and sweep
* garbage collector. * garbage collector.
@ -28,7 +28,7 @@ extern "C" {
#endif #endif
/* Global hash table to store Trackings from C/C++ /* Global hash table to store Trackings from C/C++
structs to Ruby Objects. structs to Ruby Objects.
*/ */
static st_table* swig_ruby_trackings = NULL; static st_table* swig_ruby_trackings = NULL;
@ -42,7 +42,7 @@ SWIGRUNTIME void SWIG_RubyInitializeTrackings(void) {
/* Create a hash table to store Trackings from C++ /* Create a hash table to store Trackings from C++
objects to Ruby objects. */ objects to Ruby objects. */
/* Try to see if some other .so has already created a /* Try to see if some other .so has already created a
tracking hash table, which we keep hidden in an instance var tracking hash table, which we keep hidden in an instance var
in the SWIG module. in the SWIG module.
This is done to allow multiple DSOs to share the same This is done to allow multiple DSOs to share the same
@ -101,13 +101,14 @@ SWIGRUNTIME void SWIG_RubyRemoveTracking(void* ptr) {
/* This is a helper method that unlinks a Ruby object from its /* This is a helper method that unlinks a Ruby object from its
underlying C++ object. This is needed if the lifetime of the underlying C++ object. This is needed if the lifetime of the
Ruby object is longer than the C++ object */ Ruby object is longer than the C++ object. */
SWIGRUNTIME void SWIG_RubyUnlinkObjects(void* ptr) { SWIGRUNTIME void SWIG_RubyUnlinkObjects(void* ptr) {
VALUE object = SWIG_RubyInstanceFor(ptr); VALUE object = SWIG_RubyInstanceFor(ptr);
if (object != Qnil) { if (object != Qnil) {
if (TYPE(object) != T_DATA) // object might have the T_ZOMBIE type, but that's just
abort(); // because the GC has flagged it as such for a deferred
// destruction. Until then, it's still a T_DATA object.
DATA_PTR(object) = 0; DATA_PTR(object) = 0;
} }
} }