Track the bound before invoking the callback

sferik · sferik · commit 7198fe1c3c2f · 2025-07-16T16:47:12.000-07:00
diff --git a/hash.c b/hash.c
@@ -1301,8 +1301,9 @@ hash_ar_foreach_iter(st_data_t key, st_data_t value, st_data_t argp, int error)
     if (error) return ST_STOP;
 
     ar_table *tbl = RHASH_AR_TABLE(arg->hash);
+    unsigned bound = RHASH_AR_TABLE_BOUND(arg->hash);
     int status = (*arg->func)((VALUE)key, (VALUE)value, arg->arg);
-    if (!RHASH_AR_TABLE_P(arg->hash) || RHASH_AR_TABLE(arg->hash) != tbl) {
+    if (!RHASH_AR_TABLE_P(arg->hash) || RHASH_AR_TABLE_BOUND(arg->hash) != bound) {
         rb_raise(rb_eRuntimeError, "rehash occurred during iteration");
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1301,8 +1301,9 @@ hash_ar_foreach_iter(st_data_t key, st_data_t value, st_data_t argp, int error)`
`1301`	`1301`	`if (error) return ST_STOP;`
`1302`	`1302`
`1303`	`1303`	`ar_table *tbl = RHASH_AR_TABLE(arg->hash);`
	`1304`	`+ unsigned bound = RHASH_AR_TABLE_BOUND(arg->hash);`
`1304`	`1305`	`int status = (*arg->func)((VALUE)key, (VALUE)value, arg->arg);`
`1305`		`- if (!RHASH_AR_TABLE_P(arg->hash) \|\| RHASH_AR_TABLE(arg->hash) != tbl) {`
	`1306`	`+ if (!RHASH_AR_TABLE_P(arg->hash) \|\| RHASH_AR_TABLE_BOUND(arg->hash) != bound) {`
`1306`	`1307`	`rb_raise(rb_eRuntimeError, "rehash occurred during iteration");`
`1307`	`1308`	`}`
`1308`	`1309`