1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix incorrect size of WeakMap buffer

Browse Source 
In wmap_final_func, j is the number of elements + 1 (since j also
includes the length at the 0th index), so we should resize the buffer
to size j and the new length is j - 1.
This commit is contained in:
Peter Zhu 2023-03-15 17:40:50 -04:00
parent 671ddb1eee
commit e0cf80d666
Notes: git 2023-03-16 14:00:26 +00:00 
Merged: https://github.com/ruby/ruby/pull/7536
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
weakmap.c
View File

@ -163,8 +163,8 @@ wmap_final_func(st_data_t *key, st_data_t *value, st_data_t arg, int existing)
return ST_DELETE;
}
if (j < i) {
SIZED_REALLOC_N(ptr, VALUE, j + 1, i);
ptr[0] = j;
SIZED_REALLOC_N(ptr, VALUE, j, i);
ptr[0] = j - 1;
*value = (st_data_t)ptr;
}
return ST_CONTINUE;