Fix paquettg#187

Author: paquettg

CHANGELOG.md

@@ -20,6 +20,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Added tag attribute DTO.
 - Cleaned up the selector logic.
 - Fixed issue with greedy regex for charset detection.
+- Fixed bug causing infinite loops in some cases.
 
 ### Removed
 - Curl interface and curl implementation has been removed.

src/PHPHtmlParser/Content.php

@@ -164,7 +164,7 @@ public function copyUntil(string $string, bool $char = false, bool $escape = fal
      *
      * @return string
      */
-    public function copyUntilUnless(string $string, string $unless)
+    public function copyUntilUnless(string $string, string $unless): string
     {
         $lastPos = $this->pos;
         $this->fastForward(1);

src/PHPHtmlParser/Dom.php

@@ -749,7 +749,7 @@ private function parseTag(): array
                         do {
                             $moreString = $this->content->copyUntilUnless('"', '=>');
                             $string .= $moreString;
-                        } while (!empty($moreString));
+                        } while (strlen($moreString) > 0 && $this->content->getPosition() < $this->size);
                         $attr['value'] = $string;
                         $this->content->fastForward(1);
                         $node->getTag()->setAttribute($name, $string);
@@ -760,7 +760,7 @@ private function parseTag(): array
                         do {
                             $moreString = $this->content->copyUntilUnless("'", '=>');
                             $string .= $moreString;
-                        } while (!empty($moreString));
+                        } while (strlen($moreString) > 0 && $this->content->getPosition() < $this->size);
                         $attr['value'] = $string;
                         $this->content->fastForward(1);
                         $node->getTag()->setAttribute($name, $string, false);