rallhook 0.7.0 → 0.7.1

data/CHANGELOG

@@ -1,3 +1,13 @@
+0.7.1	Fixed issue 9 ( http://github.com/tario/rallhook/issues#issue/9 ), "broken" object is processed by the GC due the impossibility of avoid 
+		the deletion of an object of class Redirect returned by handle_method. The hotfix disabled the GC at each redirection and enable
+		it again each time that a WRAPPER redirection concludes. Also the GC are re-enabled when the hook block is terminated. 
+		
+		MULTI-THREAD SUPPORT DISABLED
+		This hotfix changes a flag, the global state of GC enabled. Due this flag is not by thread, concurrency executions of hook blocks
+		in many thread simulaniously has unexpected and maybe undesirable behaviours
+		
+		The thread support will be added again to the next patch version 0.7.2
+
 0.7.0	Concurrency support, any thread can hook without affect the hook state of the other threads
 
 		Restrictions about overwrite instance methods disabled

data/README

@@ -13,12 +13,18 @@ This package contains rallhook, a ruby C extension to intercept ruby methods cal
 * distorm ( found at https://code.google.com/p/distorm/ )
 
 
-=== Gem installation
+=== Installation
+
+* run in your system
+
+sudo gem install rallhook
+
+OR
 
 * Download the last version of the gem from http://github.com/tario/ruby-cpp/downloads
 * Install the gem with the following;
 
-gem install rallhook-X.X.X.gem
+sudo gem install rallhook-X.X.X.gem
 
 == Usage
 
@@ -52,21 +58,21 @@ test.rb:
 
 standard output:
 
-	method call each:4001 over 123:Array 
-	method call print:7697 over main:Object 
-	method call write:7649 over #<IO:0x7fb15e54fad0>:IO 
-	method call to_s:3137 over 1:Fixnum 
-	1method call write:7649 over #<IO:0x7fb15e54fad0>:IO 
+	method call each:4001 over 123:Array
+	method call print:7697 over main:Object
+	method call write:7649 over #<IO:0x7fb15e54fad0>:IO
+	method call to_s:3137 over 1:Fixnum
+	1method call write:7649 over #<IO:0x7fb15e54fad0>:IO
 
-	method call print:7697 over main:Object 
-	method call write:7649 over #<IO:0x7fb15e54fad0>:IO 
-	method call to_s:3137 over 2:Fixnum 
-	2method call write:7649 over #<IO:0x7fb15e54fad0>:IO 
+	method call print:7697 over main:Object
+	method call write:7649 over #<IO:0x7fb15e54fad0>:IO
+	method call to_s:3137 over 2:Fixnum
+	2method call write:7649 over #<IO:0x7fb15e54fad0>:IO
 
-	method call print:7697 over main:Object 
-	method call write:7649 over #<IO:0x7fb15e54fad0>:IO 
-	method call to_s:3137 over 3:Fixnum 
-	3method call write:7649 over #<IO:0x7fb15e54fad0>:IO 
+	method call print:7697 over main:Object
+	method call write:7649 over #<IO:0x7fb15e54fad0>:IO
+	method call to_s:3137 over 3:Fixnum
+	3method call write:7649 over #<IO:0x7fb15e54fad0>:IO
 
 === Redirecting calls
 
@@ -92,7 +98,7 @@ standard output:
 
 			nil # do nothing
 		end
-		
+
 		def bar
 			print "bar in MethodHandler\n"
 		end
@@ -131,7 +137,7 @@ standard output:
 
 	class MethodHandler < HookHandler
 		include RallHook::Helper
-		
+
 		class FooMethodWrapper < MethodWrapper
 			def call(*x)
 				# call with reyield if block_given
@@ -139,7 +145,7 @@ standard output:
 					original_call(*x) do |*a|
 						yield(*a)
 					end
-					# add "�bar?"
+					# add "�bar?"
 					yield("bar?")
 				else
 					original_call(*x)
@@ -205,3 +211,6 @@ main.rb
 	print y.method(Y,:foo).body.file,"\n" # source2.rb
 
 
+== Copying
+
+Copyright (c) 2010 Dario Seminara, released under the GPL License (see LICENSE)

data/Rakefile

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

data/ext/rallhook_base/rallhook.c

@@ -117,6 +117,12 @@ VALUE get_hook_proc() {
 Disable the hook. Is not usually necesary because of the RAII feature of Hook#hook
 */
 VALUE unhook(VALUE self) {
+	disable_redirect(tinfo_from_thread( rb_thread_current() ) );
+	rb_gc_enable();
+	return Qnil;
+}
+
+VALUE restore_unhook(VALUE self) {
 	disable_redirect(tinfo_from_thread( rb_thread_current() ) );
 	return Qnil;
 }
@@ -191,6 +197,9 @@ void rallhook_redirect_handler ( VALUE* klass, VALUE* recv, ID* mid ) {
 
 		// method named "binding" cannot be redirected
 		if (rb_obj_is_kind_of(result,rb_mMethodRedirect) == Qtrue ) {
+
+			rb_gc_disable();
+
 			*klass = rb_ivar_get(result,id_klass_var );
 			*recv = rb_ivar_get(result,id_recv_var );
 			*mid = rb_to_id( rb_ivar_get(result,id_method_var) );
@@ -257,7 +266,7 @@ If no call to Hook#hook has made, rehook does nothing
 VALUE rehook(VALUE unused) {
 	enable_redirect(tinfo_from_thread( rb_thread_current() ));
 	if (rb_block_given_p() ) {
-		return rb_ensure(rb_yield, Qnil, unhook, Qnil);
+		return rb_ensure(rb_yield, Qnil, restore_unhook, Qnil);
 	}
 	return Qnil;
 }

data/lib/rallhook.rb

@@ -40,7 +40,7 @@ module RallHook
 #
   class Redirect
 		include MethodRedirect
-		
+
 		attr_reader :recv
 
 		def initialize(klass, recv, m, unhook = nil)
@@ -225,7 +225,8 @@ module RallHook
           call(*args)
         end
       ensure
-        rehook
+        GC.enable
+        ::RallHook::Hook.rehook
       end
 
 #
@@ -325,6 +326,11 @@ class Object
 # Anyway, it is desirable to use MethodWrapper instead to "wrap" method calls
 #
 # see RallHook::Helper::MethodWrapper
+#
+# NOTE: Rallhook internally disable the GC in a redirection, you must reactivate it after
+#
+# NOTE 2: If use the MethodWrapper functionallity, it reactivates the GC disabled by rallhook. You should not
+# worry about it
 #
   def redirect_with_unhook(method_name, klass = nil)
     if klass
@@ -358,6 +364,7 @@ class Object
 #
 #   # hook using MethodHandler, etc... (see README and examples)
 #
+# NOTE: Rallhook internally disable the GC in a redirection, you must reactivate it after
 #
   def redirect(method_name, klass = nil)
 		if klass

metadata

@@ -1,7 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rallhook
 version: !ruby/object:Gem::Version 
-  version: 0.7.0
+  hash: 1
+  prerelease: false
+  segments: 
+  - 0
+  - 7
+  - 1
+  version: 0.7.1
 platform: ruby
 authors: 
 - Dario Seminara
@@ -9,19 +15,25 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-06-20 00:00:00 -03:00
+date: 2010-06-21 00:00:00 -03:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: ruby-cymbol
-  type: :runtime
-  version_requirement: 
-  version_requirements: !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 27
+        segments: 
+        - 0
+        - 1
+        - 0
         version: 0.1.0
-    version: 
+  type: :runtime
+  version_requirements: *id001
 description: 
 email: robertodarioseminara@gmail.com
 executables: []
@@ -91,21 +103,27 @@ rdoc_options:
 require_paths: 
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
-  version: 
 required_rubygems_version: !ruby/object:Gem::Requirement 
+  none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
-  version: 
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.3.5
+rubygems_version: 1.3.7
 signing_key: 
 specification_version: 3
 summary: Allow hooking of all method invocations transparently to control and / or monitor the behavior of a ruby program