Incorrect error line numbers reported in Dom\HTMLDocument::createFromString #17486

xPaw · 2025-01-16T15:31:45Z

Description

<?php

$repeated = str_repeat('a', 50000);

$html = <<<HTML
<!DOCTYPE html>
<html lang="en">
	<body>
		<svg>
			<path d="{$repeated}" />
		</svg>
		<div>&#xa;</div>
	</body>
</html>
HTML;

// this reports tokenizer error control-character-reference in Entity, line: 53, column: 883
$Document = \Dom\HTMLDocument::createFromString($html);

This only happens for tokenizer errors, but not tree errors: change 
 to <hr /> and it will correctly report line: 7, column: 9-10.

PHP Version

8.4.3

Operating System

No response

The text was updated successfully, but these errors were encountered:

…ument::createFromString

* PHP-8.4: Fix GH-17481: UTF-8 corruption in \Dom\HTMLDocument Fix GH-17486: Incorrect error line numbers reported in Dom\HTMLDocument::createFromString

xPaw added Bug Status: Needs Triage labels Jan 16, 2025

nielsdos added Extension: dom Status: Verified and removed Status: Needs Triage labels Jan 16, 2025

nielsdos self-assigned this Jan 16, 2025

nielsdos linked a pull request Jan 16, 2025 that will close this issue

Fix GH-17486: Incorrect error line numbers reported in Dom\HTMLDocument::createFromString #17491

Closed

nielsdos closed this as completed in 21c170c Jan 17, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Incorrect error line numbers reported in Dom\HTMLDocument::createFromString #17486

Incorrect error line numbers reported in Dom\HTMLDocument::createFromString #17486

xPaw commented Jan 16, 2025 •

edited

Loading

Incorrect error line numbers reported in Dom\HTMLDocument::createFromString #17486

Incorrect error line numbers reported in Dom\HTMLDocument::createFromString #17486

Comments

xPaw commented Jan 16, 2025 • edited Loading

Description

PHP Version

Operating System

xPaw commented Jan 16, 2025 •

edited

Loading