From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sat, 23 Mar 2019 21:40:19 +0000 (-0400)
Subject: Remove inadequate check for duplicate "xml" PI.
X-Git-Tag: REL9_6_13~47
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=87a44b6cf98f8d325af0f000f10adedcb7a5b95d;p=postgresql

Remove inadequate check for duplicate "xml" PI.

I failed to think about PIs starting with "xml".  We don't really
need this check at all, so just take it out.  Oversight in
commit 8d1dadb25 et al.
---

diff --git a/src/backend/utils/adt/xml.c b/src/backend/utils/adt/xml.c
index 7dd06c7bc6..62db7b2e8f 100644
--- a/src/backend/utils/adt/xml.c
+++ b/src/backend/utils/adt/xml.c
@@ -1400,10 +1400,6 @@ xml_doctype_in_content(const xmlChar *str)
 		if (!e)
 			return false;
 
-		/* we don't check PIs carefully, but do reject "xml" target */
-		if (e - p >= 3 && xmlStrncasecmp(p, (xmlChar *) "xml", 3) == 0)
-			return false;
-
 		/* advance over PI, keep scanning */
 		p = e + 2;
 	}