From 418ab58f94ea4a0957bcad8e7520bb3fb62d1394 Mon Sep 17 00:00:00 2001
From: Leonard Richardson <leonardr@segfault.org>
Date: Sun, 4 Jun 2023 18:44:29 -0400
Subject: Fixed a case found by Mengyuhan where html.parser giving up on  
 markup would result in an AssertionError instead of a   ParserRejectedMarkup
 exception.

---
 bs4/builder/_htmlparser.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'bs4/builder/_htmlparser.py')

diff --git a/bs4/builder/_htmlparser.py b/bs4/builder/_htmlparser.py
index e065096..3cc187f 100644
--- a/bs4/builder/_htmlparser.py
+++ b/bs4/builder/_htmlparser.py
@@ -378,10 +378,10 @@ class HTMLParserTreeBuilder(HTMLTreeBuilder):
         parser.soup = self.soup
         try:
             parser.feed(markup)
+            parser.close()
         except AssertionError as e:
             # html.parser raises AssertionError in rare cases to
             # indicate a fatal problem with the markup, especially
             # when there's an error in the doctype declaration.
             raise ParserRejectedMarkup(e)
-        parser.close()
         parser.already_closed_empty_element = []
-- 
cgit v1.2.3