Fix IndexOutOfBounds when parsing a wrong Html (#22)

* Fix issue with encoding some html entities * Fix issues with decoding emoji * Fix IndexOutOfBounds when parsing a wrong Html
MohamedRejeb · Sep 1, 2023 · 5d88c58 · 5d88c58
1 parent 3f0ce25
commit 5d88c58
Show file tree

Hide file tree

Showing 3 changed files with 763 additions and 1 deletion.
diff --git a/ksoup-html/src/commonMain/kotlin/com/mohamedrejeb/ksoup/html/parser/KsoupHtmlParser.kt b/ksoup-html/src/commonMain/kotlin/com/mohamedrejeb/ksoup/html/parser/KsoupHtmlParser.kt
@@ -184,7 +184,7 @@ public class KsoupHtmlParser(
         this.endOpenTag(isOpenImplied)
 
         // Self-closing tags will be on the top of the stack
-        if (this.stack[this.stack.size - 1] == name) {
+        if (this.stack.getOrNull(this.stack.size - 1) == name) {
             // If the opening tag isn't implied, the closing tag has to be implied.
             this.handler.onCloseTag(name, !isOpenImplied)
             this.stack.removeLast()

diff --git a/ksoup-html/src/commonTest/kotlin/com/mohamedrejeb/ksoup/html/KsoupHtmlParserTest.kt b/ksoup-html/src/commonTest/kotlin/com/mohamedrejeb/ksoup/html/KsoupHtmlParserTest.kt
@@ -270,4 +270,17 @@ class KsoupHtmlParserTest {
             expectedString = "First lineSecond line",
         )
     }
+
+    @Test
+    fun testBigHtml() {
+        val handler = KsoupHtmlHandler
+            .Builder()
+            .build()
+
+        val ksoupHtmlParser = KsoupHtmlParser(
+            handler = handler,
+        )
+        ksoupHtmlParser.write(TestData.htmlString)
+        ksoupHtmlParser.end()
+    }
 }