bdnugget/php-src
1
0 Fork 0
mirror of https://github.com/bdnugget/php-src.git synced 2026-11-07 17:27:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix garbage marking in gc_collect_roots()

Browse Source 
gc_collect_white() will mark white nodes as black and add them as
garbage, but only if it's not buffered yet. The already buffered
roots are instead marked as garbage in gc_collect_roots() directly.
However, if gc_collect_white() marked a (buffered) root as black
through recursion, it would not subsequently be marked as garbage.
This commit is contained in:
Nikita Popov committed 2018-03-07 09:50:25 +01:00
1 parent dcbac8ade6
commit 3b5b64ce75
1 file changed
+4 -5
Download Patch File Download Diff File Expand all files Collapse all files
Zend/zend_gc.c
+4 -5
View File
@@ -1167,11 +1167,10 @@ static int gc_collect_roots(uint32_t *flags)
while (idx != end) {
current = GC_IDX2PTR(idx);
ref = current->ref;
if (GC_IS_ROOT(ref)) {
if (GC_REF_CHECK_COLOR(ref, GC_WHITE)) {
current->ref = GC_MAKE_GARBAGE(ref);
count += gc_collect_white(ref, flags);
}
ZEND_ASSERT(GC_IS_ROOT(ref));
current->ref = GC_MAKE_GARBAGE(ref);
if (GC_REF_CHECK_COLOR(ref, GC_WHITE)) {
count += gc_collect_white(ref, flags);
}
idx++;
}