Merge pull request #1 from priyadi/master

throw an exception when server doesn't give us list of mechanisms
candy-chat · Oct 22, 2013 · ea46d9c · ea46d9c
2 parents 597cada + 21abba8
commit ea46d9c
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/lib/XmppPrebind.php b/lib/XmppPrebind.php
@@ -133,7 +133,12 @@ public function connect($username, $password) {
 		$this->sid = $body->getAttribute('sid');
 		$this->debug($this->sid, 'sid');
 
-		$mechanisms = $body->firstChild->firstChild->getElementsByTagName('mechanism');
+		$child = $body->firstChild->firstChild;
+		if (is_object($child)) {
+			$mechanisms = $child->getElementsByTagName('mechanism');
+		} else {
+			throw new Exception('Invalid response');
+		}
 
 		foreach ($mechanisms as $value) {
 			$this->mechanisms[] = $value->nodeValue;
@@ -617,4 +622,4 @@ protected function getAndIncrementRid() {
  */
 class XmppPrebindException extends Exception{}
 
-class XmppPrebindConnectionException extends XmppPrebindException {}
+class XmppPrebindConnectionException extends XmppPrebindException {}