Fixed a crash when opening broken xml file

lib/irrlicht/source/Irrlicht/CXMLReaderImpl.h

@@ -225,7 +225,7 @@ private:
 		char_type* start = P;
 
 		// more forward until '<' found
-		while(*P != L'<' && *P)
+		while(*P && *P != L'<')
 			++P;
 
 		// not a node, so return false
@@ -239,6 +239,7 @@ private:
 				return true;
 		}
 
+		if (*P)
 			++P;
 
 		// based on current token, parse and report next element
@@ -297,9 +298,10 @@ private:
 		CurrentNodeType = EXN_UNKNOWN;
 
 		// move until end marked with '>' reached
-		while(*P != L'>')
+		while(*P && *P != L'>')
 			++P;
 
+		if (*P)
 			++P;
 	}
 
@@ -308,6 +310,7 @@ private:
 	void parseComment()
 	{
 		CurrentNodeType = EXN_COMMENT;
+		if (*P)
 			P += 1;
 
 		char_type *pCommentBegin = P;
@@ -315,7 +318,7 @@ private:
 		int count = 1;
 
 		// move until end of comment reached
-		while(count)
+		while(*P && count)
 		{
 			if (*P == L'>')
 				--count;
@@ -326,10 +329,17 @@ private:
 			++P;
 		}
 
+		if (*P)
+		{
 			P -= 3;
 			NodeName = core::string<char_type>(pCommentBegin+2, (int)(P - pCommentBegin-2));
 			P += 3;
 		}
+		else
+		{
+			NodeName = "";
+		}
+	}
 
 
 	//! parses an opening xml element and reads attributes
@@ -343,13 +353,13 @@ private:
 		const char_type* startName = P;
 
 		// find end of element
-		while(*P != L'>' && !isWhiteSpace(*P))
+		while(*P && *P != L'>' && !isWhiteSpace(*P))
 			++P;
 
 		const char_type* endName = P;
 
 		// find Attributes
-		while(*P != L'>')
+		while(*P && *P != L'>')
 		{
 			if (isWhiteSpace(*P))
 				++P;
@@ -362,15 +372,16 @@ private:
 					// read the attribute names
 					const char_type* attributeNameBegin = P;
 
-					while(!isWhiteSpace(*P) && *P != L'=')
+					while(*P && !isWhiteSpace(*P) && *P != L'=')
 						++P;
 
 					const char_type* attributeNameEnd = P;
+					if (*P)
 						++P;
 
 					// read the attribute value
 					// check for quotes and single quotes, thx to murphy
-					while( (*P != L'\"') && (*P != L'\'') && *P)
+					while(*P && (*P != L'\"') && (*P != L'\''))
 						++P;
 
 					if (!*P) // malformatted xml file
@@ -378,16 +389,18 @@ private:
 
 					const char_type attributeQuoteChar = *P;
 
+					if (*P)
 						++P;
 					const char_type* attributeValueBegin = P;
 
-					while(*P != attributeQuoteChar && *P)
+					while(*P && *P != attributeQuoteChar)
 						++P;
 
 					if (!*P) // malformatted xml file
 						return;
 
 					const char_type* attributeValueEnd = P;
+					if (*P)
 						++P;
 
 					SAttribute attr;
@@ -403,6 +416,7 @@ private:
 				else
 				{
 					// tag is closed directly
+					if (*P)
 						++P;
 					IsEmptyElement = true;
 					break;
@@ -420,6 +434,7 @@ private:
 
 		NodeName = core::string<char_type>(startName, (int)(endName - startName));
 
+		if (*P)
 			++P;
 	}
 
@@ -431,20 +446,23 @@ private:
 		IsEmptyElement = false;
 		Attributes.clear();
 
+		if (*P)
 			++P;
 		const char_type* pBeginClose = P;
 
-		while(*P != L'>')
+		while(*P && *P != L'>')
 			++P;
 
 		NodeName = core::string<char_type>(pBeginClose, (int)(P - pBeginClose));
+		
+		if (*P)
 			++P;
 	}
 
 	//! parses a possible CDATA section, returns false if begin was not a CDATA section
 	bool parseCDATA()
 	{
-		if (*(P+1) != L'[')
+		if (!*P || !*(P+1) || *(P+1) != L'[')
 			return false;
 
 		CurrentNodeType = EXN_CDATA;