From 5fa36a640168d3eaf27f3d6630ffd71527bcd724 Mon Sep 17 00:00:00 2001
From: wltb <wltb@localhost.com>
Date: Mon, 9 Sep 2013 00:44:55 +0200
Subject: [PATCH 1/2] Feedparser: Change handling of libxml error 9 (cycle all
 errors)

---
 classes/feedparser.php | 37 ++++++++++++++++++++-----------------
 1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/classes/feedparser.php b/classes/feedparser.php
index 4a2c6c2da..22052bdb6 100644
--- a/classes/feedparser.php
+++ b/classes/feedparser.php
@@ -23,18 +23,12 @@ class FeedParser {
 		$error = libxml_get_last_error();
 
 		// libxml compiled without iconv?
-		if ($error && ($error->code == 32 || $error->code == 9)) {
-			if (preg_match('/^(<\?xml[\t\n\r ].*?encoding=["\'])(.+?)(["\'].*?\?>)/s', $data, $matches) === 1) {
-				$enc = $matches[2];
-
-				$data = mb_convert_encoding($data, 'UTF-8', $enc);
+		if ($error && $error->code == 32) {
+			if (preg_match('/^(<\?xml[\t\n\r ].*?encoding[\t\n\r ]*=[\t\n\r ]*["\'])(.+?)(["\'].*?\?>)/s', $data, $matches) === 1) {
+				$data = mb_convert_encoding($data, 'UTF-8', $matches[2]);
 
 				$data = preg_replace('/^<\?xml[\t\n\r ].*?\?>/s', $matches[1] . "UTF-8" . $matches[3] , $data);
 
-
-				// apparently not all UTF-8 characters are valid for XML
-				$data = preg_replace('/[^\x{0009}\x{000a}\x{000d}\x{0020}-\x{D7FF}\x{E000}-\x{FFFD}]+/u', ' ', $data);
-
 				if ($data) {
 					libxml_clear_errors();
 
@@ -43,20 +37,29 @@ class FeedParser {
 
 					$error = libxml_get_last_error();
 				}
-		   }
+			}
 		}
 
 		// some terrible invalid unicode entity?
-		if ($error && $error->code == 9) {
-			$data = mb_convert_encoding($data, 'UTF-8', 'UTF-8');
+		if ($error) {
+			foreach(libxml_get_errors() as $err) {
+				if ($err->code == 9) {
+					// remove dangling bytes
+					$data = mb_convert_encoding($data, 'UTF-8', 'UTF-8');
+					
+					// apparently not all UTF-8 characters are valid for XML
+					$data = preg_replace('/[^\x{0009}\x{000a}\x{000d}\x{0020}-\x{D7FF}\x{E000}-\x{FFFD}]+/u', ' ', $data);
 
-			if ($data) {
-				libxml_clear_errors();
+					if ($data) {
+						libxml_clear_errors();
 
-				$this->doc = new DOMDocument();
-				$this->doc->loadXML($data);
+						$this->doc = new DOMDocument();
+						$this->doc->loadXML($data);
 
-				$error = libxml_get_last_error();
+						$error = libxml_get_last_error();
+					}
+					break;
+				}
 			}
 		}
 

From d3305ff8de6ca9881021646221827d27240dba38 Mon Sep 17 00:00:00 2001
From: wltb <wltb@localhost.com>
Date: Tue, 24 Sep 2013 16:43:47 +0200
Subject: [PATCH 2/2] Feedparser: Only format fatal errors

---
 classes/feedparser.php | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/classes/feedparser.php b/classes/feedparser.php
index 22052bdb6..07de73a9d 100644
--- a/classes/feedparser.php
+++ b/classes/feedparser.php
@@ -62,8 +62,16 @@ class FeedParser {
 				}
 			}
 		}
-
-		$this->error = $this->format_error($error);
+		
+		$this->error = "";
+		if($error) {
+			foreach(libxml_get_errors() as $error) {
+				if($error->level == LIBXML_ERR_FATAL) {
+					$this->error = $this->format_error($error);
+					break; //break here because currently we only show one error
+				}
+			}	
+		}
 		libxml_clear_errors();
 
 		$this->items = array();