[dav] have GetStatusTag() iterate over XML elements, not nodes.

As things are, if there are no children of the passed element, we'll have a NULL dereference.

diff --git a/xbmc/filesystem/DAVCommon.cpp b/xbmc/filesystem/DAVCommon.cpp
index 179e98c..6fdb35d 100644 (file)
--- a/xbmc/filesystem/DAVCommon.cpp
+++ b/xbmc/filesystem/DAVCommon.cpp
@@ -70,11 +70,11 @@ std::string CDAVCommon::GetStatusTag(const TiXmlElement *pElement)
 {
   const TiXmlElement *pChild;
 
-  for (pChild = pElement->FirstChild()->ToElement(); pChild != 0; pChild = pChild->NextSibling()->ToElement())
+  for (pChild = pElement->FirstChildElement(); pChild != 0; pChild = pChild->NextSiblingElement())
   {
     if (ValueWithoutNamespace(pChild, "status"))
     {
-      return pChild->GetText();
+      return pChild->NoChildren() ? "" : pChild->FirstChild()->ValueStr();
     }
   }