rallhook 0.7.4 → 0.7.5

data/CHANGELOG

@@ -1,3 +1,5 @@
+0.7.5	Fixed issue 13 (Segmentation fault in ruby 1.9)
+
 0.7.4	Fixed issue 12 (Compile error for ruby 1.9)
 
 0.7.3	Fixed issue 10 (Compile Error in i486 Arch (-ldistorm64 not found) )

data/Rakefile

@@ -6,7 +6,7 @@ require 'rake/gempackagetask'
 
 spec = Gem::Specification.new do |s|
   s.name = 'rallhook'
-  s.version = '0.7.4'
+  s.version = '0.7.5'
   s.author = 'Dario Seminara'
   s.email = 'robertodarioseminara@gmail.com'
   s.platform = Gem::Platform::RUBY

data/ext/rallhook_base/rallhook.c

@@ -241,6 +241,8 @@ VALUE hook(VALUE self, VALUE hook_proc) {
 
 	put_redirect_handler( rallhook_redirect_handler );
 
+	hook_rb_add_method();
+
 	enable_redirect(tinfo_from_thread(rb_thread_current()));
 
 	if (rb_block_given_p() ) {

data/ext/rallhook_base/restrict_def.c

@@ -118,7 +118,13 @@ void rb_add_method_fake(
 		if (FL_TEST(klass, FL_SINGLETON)) {
     		// singleton method over classes are illegal
 			if ( strcmp( rb_class2name(klass), "Class") == 0) {
+
+#ifdef RUBY1_9
+				st_data_t result;
+#endif
+#ifdef RUBY1_8
 				int result;
+#endif
 				if (st_lookup(RCLASS(klass)->m_tbl,id,&result) ) {
 				rb_raise(rb_eSecurityError, "Illegal overwrite of singleton method %s", rb_id2name(id) );
 				}
@@ -131,7 +137,6 @@ void rb_add_method_fake(
 	} else {
 		rb_add_method_copy(klass,id,node,noex);
 	}
-
 }
 
 void shadow_redirect(VALUE* klass, VALUE* recv, ID* mid) {
@@ -144,9 +149,7 @@ void shadow_redirect(VALUE* klass, VALUE* recv, ID* mid) {
 
 int add_method_code_changed = 0;
 
-void init_restrict_def() {
-
-
+void hook_rb_add_method() {
 	if (!add_method_code_changed) {
 
 		void* handle = dlopen(current_libruby(),0x101);
@@ -163,6 +166,9 @@ void init_restrict_def() {
 
 		add_method_code_changed = 1;
 	}
+}
+
+void init_restrict_def() {
 
 	__shadow___id = rb_intern("__shadow__");
 	__unshadow___id = rb_intern("__unshadow__");

data/ext/rallhook_base/restrict_def.h

@@ -34,4 +34,6 @@ VALUE unshadow(VALUE klass) ;
 
 VALUE shadow_or_create(VALUE klass);
 
+void hook_rb_add_method();
+
 #endif

data/ext/rallhook_base/ruby_redirect.c

@@ -100,8 +100,9 @@ pthread_mutex_t put_redirect_handler_mutex;
 int put_redirect_handler( REDIRECTHANDLER redirect_handler) {
 
 	pthread_mutex_lock(&put_redirect_handler_mutex);
-	put_redirect_handler_(redirect_handler);
+	int ret = put_redirect_handler_(redirect_handler);
 	pthread_mutex_unlock(&put_redirect_handler_mutex);
+	return ret;
 }
 
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rallhook
 version: !ruby/object:Gem::Version 
-  hash: 11
+  hash: 9
   prerelease: false
   segments: 
   - 0
   - 7
-  - 4
-  version: 0.7.4
+  - 5
+  version: 0.7.5
 platform: ruby
 authors: 
 - Dario Seminara
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-07-23 00:00:00 -03:00
+date: 2010-08-07 00:00:00 -03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency