memory-profiler 1.1.14 → 1.1.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c26d5bc8b4215de3776e42ec9cd8daaf54a3cba9d55449f6b570ba7ee5ef9c91
-  data.tar.gz: 6668320402b35e3633eff016bd79b34c6b69952cb03120fc7f04886782bacfe3
+  metadata.gz: cce184966c5eb2b104059f419fb7f220b2bea168b5359adacc90923079332241
+  data.tar.gz: 436690f9e75f5ba72546604fd58d71efccb56d4dc8b8880feaa3b4ecab03c918
 SHA512:
-  metadata.gz: 68404ae44a442584084727cc854ed81abc36964c5621b93361cde70c6ae80f71ad21c7a45caf049c5906846cec8dff23ccbddb2e021eedf137e8ba7d57c78363
-  data.tar.gz: 406d8caac175e30907d752cec211d066a8c7ed5bf260c4e4eba6304f73b60d7c47be6d68d8b6cc4e738cb8fe19ebadeea0f251c7b2df888923b86abdaae2867e
+  metadata.gz: 54d4d1e2f8d19da788e53ffa0c5e69dcaf5db708d5d1de0336bccc79c6311330a56b0c8cb605b7bb7dbc116b91b91a752cacf317d4403e5188517ddcdded12a2
+  data.tar.gz: 4994f81f0b6dd2b0c46bc3aa3e415dbc16ffb633d9fefdeafd67d10df522aaccc5003dd632a664b8eb69e476053bac5f51d0979c69c4daa97965356b1bb2ca9c

data/ext/memory/profiler/capture.c

@@ -298,21 +298,6 @@ static void Memory_Profiler_Capture_event_callback(VALUE data, void *ptr) {
 	VALUE klass = rb_class_of(object);
 	if (!klass) return;
 	
-	// Check if class is already freed (T_NONE). This can happen when:
-	// 1. Object was allocated before capture started (no NEWOBJ event queued).
-	// 2. Anonymous class (Class.new) was freed before its instances.
-	//
-	// If the class is T_NONE, it means:
-	// - It's NOT in tracked_classes (would retain it via GC mark callback).
-	// - It's NOT in any pending NEWOBJ event (would retain it via event queue marking).
-	//
-	// Therefore process_freeobj() would skip this event anyway (class lookup fails at line 200).
-	// We must skip enqueueing to avoid attempting to mark a T_NONE object during GC,
-	// which can cause: [BUG] try to mark T_NONE object.
-	if (rb_type(klass) == T_NONE) {
-		return;
-	}
-	
 	if (DEBUG) {
 		const char *klass_name = "(ignored)";
 		if (event_flag == RUBY_INTERNAL_EVENT_NEWOBJ) {
@@ -325,6 +310,7 @@ static void Memory_Profiler_Capture_event_callback(VALUE data, void *ptr) {
 		// Skip NEWOBJ if disabled (during callback) to prevent infinite recursion
 		if (capture->paused) return;
 		
+		// Check if class is already freed (T_NONE). This can happen when:
 		// It's safe to unconditionally call here:
 		object = rb_obj_id(object);
 
@@ -332,6 +318,10 @@ static void Memory_Profiler_Capture_event_callback(VALUE data, void *ptr) {
 	} else if (event_flag == RUBY_INTERNAL_EVENT_FREEOBJ) {
 		// We only care about objects that have been seen before (i.e. have an object ID):
 		if (RB_FL_TEST(object, FL_SEEN_OBJ_ID)) {
+			// For freeobj, we only care about klasses that we are tracking.
+			// This prevents us from enqueuing klass objects that might be freed.
+			if (!st_lookup(capture->tracked_classes, (st_data_t)klass, NULL)) return;
+			
 			// It's only safe to call here if the object already has an object ID.
 			object = rb_obj_id(object);
 			

data/lib/memory/profiler/version.rb

@@ -7,7 +7,7 @@
 module Memory
 	# @namespace
 	module Profiler
-		VERSION = "1.1.14"
+		VERSION = "1.1.15"
 	end
 end
 

data/readme.md

@@ -22,6 +22,10 @@ Please see the [project documentation](https://socketry.github.io/memory-profile
 
 Please see the [project releases](https://socketry.github.io/memory-profiler/releases/index) for all releases.
 
+### v1.1.15
+
+  - Ignore `freeobj` for classes that are not being tracked.
+
 ### v1.1.14
 
   - Ignore `freeobj` events for objects with anonymous classes that are not tracked (and thus become `T_NONE`).
@@ -66,10 +70,6 @@ Please see the [project releases](https://socketry.github.io/memory-profiler/rel
   - Better state handling and object increment/decrement counting.
   - Better call tree handling - including support for `prune!`.
 
-### v1.1.5
-
-  - Use queue for `newobj` too to avoid invoking user code during object allocation.
-
 ## Contributing
 
 We welcome contributions to this project.

data/releases.md

@@ -1,5 +1,9 @@
 # Releases
 
+## v1.1.15
+
+  - Ignore `freeobj` for classes that are not being tracked.
+
 ## v1.1.14
 
   - Ignore `freeobj` events for objects with anonymous classes that are not tracked (and thus become `T_NONE`).

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: memory-profiler
 version: !ruby/object:Gem::Version
-  version: 1.1.14
+  version: 1.1.15
 platform: ruby
 authors:
 - Samuel Williams