From: Gavin Sherry <swm@php.net>
Date: Wed, 22 Aug 2001 02:03:14 +0000 (+0000)
Subject: Changed php_strip_tags() to check if <? was XML code.
X-Git-Tag: PRE_SUBST_Z_MACROS~400
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ff7a694e0f020fda7f15e14b2f7ba9f04325f2a9;p=php

Changed php_strip_tags() to check if <? was XML code.
---

diff --git a/ext/standard/string.c b/ext/standard/string.c
index 9ef011d51e..9656b116ca 100644
--- a/ext/standard/string.c
+++ b/ext/standard/string.c
@@ -3190,6 +3190,9 @@ int php_tag_find(char *tag, int len, char *set) {
 	When an allow string is passed in we keep track of the string
 	in state 1 and when the tag is closed check it against the
 	allow string to see if we should allow it.
+
+	swm: Added ability to strip <?xml tags without assuming it PHP
+	code.
 */
 PHPAPI void php_strip_tags(char *rbuf, int len, int state, char *allow, int allow_len)
 {
@@ -3286,13 +3289,18 @@ PHPAPI void php_strip_tags(char *rbuf, int len, int state, char *allow, int allo
 				break;
 
 			case '?':
-				if (state==1 && *(p-1)=='<') {
+				if (state==1 && *(p-1)=='<' && *(p+1) != 'x' 
+				  && *(p+2) != 'm' && *(p+3) != 'l') {
+
 					br=0;
 					state=2;
 					break;
 				}
-				/* fall-through */
+					/* else, it is xml, since state == 1, lets just fall through
+					 * to '>'
+					 */
 
+				/* fall-through */
 			default:
 				if (state == 0) {
 					*(rp++) = c;
@@ -3301,7 +3309,7 @@ PHPAPI void php_strip_tags(char *rbuf, int len, int state, char *allow, int allo
 					if( (tp-tbuf)>=PHP_TAG_BUF_SIZE ) { /* no buffer overflows */
 						tp = tbuf;
 					}
-				}
+				} 
 				break;
 		}
 		c = *(++p);