From 5e945c516995c8a7d9a53f865275b7d3fd221bd0 Mon Sep 17 00:00:00 2001
From: Andreas Kling <kling@serenityos.org>
Date: Sat, 17 Apr 2021 21:36:37 +0200
Subject: [PATCH] LibWeb: Don't load anything for <iframe> without src
 attribute

Completing an empty URL string from the document base URL will just
return the document URL, so any document that had an "<iframe>"
would endlessly load itself in recursive iframes.
---
 Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.cpp b/Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.cpp
index 95efd33ca6..c607e0be1c 100644
--- a/Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.cpp
+++ b/Userland/Libraries/LibWeb/HTML/HTMLIFrameElement.cpp
@@ -66,6 +66,9 @@ void HTMLIFrameElement::load_src(const String& value)
     if (!m_content_frame)
         return;
 
+    if (value.is_null())
+        return;
+
     auto url = document().complete_url(value);
     if (!url.is_valid()) {
         dbgln("iframe failed to load URL: Invalid URL: {}", value);