Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,10 @@ PHP NEWS
when lexbor allocation fails). (ndossche)
. Fixed bug GH-25434 (C14NFile() on a Dom\XMLDocument leaves the document
changed and leaks memory). (lazerg)
. Fixed CSS selectors checking only the nearest element for a descendant or
subsequent-sibling combinator after a compound selector that starts with
:is(), :where(), :not(), :has() or :nth-child(An+B of S), by merging
lexbor/lexbor@f6c1d06. (phcorp)

- GD:
. Fixed use-after-free when drawing with a brush or tile image that was
Expand Down
27 changes: 27 additions & 0 deletions ext/dom/lexbor/lexbor/selectors-adapted/selectors.c
Original file line number Diff line number Diff line change
Expand Up @@ -189,6 +189,10 @@ static lxb_selectors_entry_t *
lxb_selectors_state_find(lxb_selectors_t *selectors,
lxb_selectors_entry_t *entry);

static lxb_selectors_entry_t *
lxb_selectors_state_find_forward(lxb_selectors_t *selectors,
lxb_selectors_entry_t *entry);

static lxb_selectors_entry_t *
lxb_selectors_state_found_check(lxb_selectors_t *selectors,
lxb_selectors_entry_t *entry);
Expand Down Expand Up @@ -673,6 +677,29 @@ lxb_inline void
lxb_selectors_switch_to_not_found(lxb_selectors_t *selectors,
lxb_selectors_nested_t *current)
{
lxb_css_selector_combinator_t combinator;

combinator = current->entry->combinator;

/*
* A nested selector rejected this candidate. Continue the combinator
* before backtracking to another entry.
*/
if (combinator == LXB_CSS_SELECTOR_COMBINATOR_DESCENDANT
|| combinator == LXB_CSS_SELECTOR_COMBINATOR_FOLLOWING
|| (current->forward
&& combinator == LXB_CSS_SELECTOR_COMBINATOR_CHILD))
{
if (current->forward) {
selectors->state = lxb_selectors_state_find_forward;
}
else {
selectors->state = lxb_selectors_state_find;
}

return;
}

if (current->forward) {
selectors->state = lxb_selectors_state_not_found_forward;
}
Expand Down
43 changes: 43 additions & 0 deletions ext/dom/tests/modern/css_selectors/lexbor436.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,43 @@
--TEST--
Lexbor #436 (combinator after a compound-leading selector-taking pseudo-class checks only one element)
--EXTENSIONS--
dom
--FILE--
<?php

$siblings = Dom\HTMLDocument::createFromString(
'<!DOCTYPE html><div><x></x><y></y><z></z><w></w></div>'
);
$nested = Dom\HTMLDocument::createFromString(
'<!DOCTYPE html><ul><li>a<ol><li>b</li></ol></li></ul>'
);

$cases = [
[$siblings, 'div > :is(y) ~ *'],
[$siblings, 'div > :where(y) ~ *'],
[$nested, ':is(ul) li'],
[$nested, ':where(ul) li'],
[$nested, ':not(ol) li'],
[$nested, ':has(> li > ol) li'],
[$nested, ':nth-child(1 of ul) li'],
[$nested, 'body :is(li) li'],
];

foreach ($cases as [$dom, $selector]) {
$result = [];
foreach ($dom->querySelectorAll($selector) as $node) {
$result[] = $node->localName . '(' . trim($node->firstChild?->textContent ?? '') . ')';
}
echo $selector, ': ', implode(' ', $result), "\n";
}

?>
--EXPECT--
div > :is(y) ~ *: z() w()
div > :where(y) ~ *: z() w()
:is(ul) li: li(a) li(b)
:where(ul) li: li(a) li(b)
:not(ol) li: li(a) li(b)
:has(> li > ol) li: li(a) li(b)
:nth-child(1 of ul) li: li(a) li(b)
body :is(li) li: li(b)
Loading