remoting: Fix response validation error found by PVS Studio.

remoting/jingle_glue/iq_request.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "remoting/jingle_glue/iq_request.h"
 
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/string_number_conversions.h"
@@ skipping 227 matching lines @@
   }
 
   const buzz::XmlElement* stun = query->FirstNamed(buzz::QN_JINGLE_INFO_STUN);
   if (stun) {
     for (const buzz::XmlElement* server =
          stun->FirstNamed(buzz::QN_JINGLE_INFO_SERVER);
          server != NULL;
          server = server->NextNamed(buzz::QN_JINGLE_INFO_SERVER)) {
       std::string host = server->Attr(buzz::QN_JINGLE_INFO_HOST);
       std::string port_str = server->Attr(buzz::QN_JINGLE_INFO_UDP);
-      if (host != buzz::STR_EMPTY && host != buzz::STR_EMPTY) {
+      if (host != buzz::STR_EMPTY && port_str != buzz::STR_EMPTY) {
         int port;
         if (!base::StringToInt(port_str, &port)) {
           LOG(WARNING) << "Unable to parse port in stanza" << stanza->Str();
         } else {
           stun_hosts.push_back(talk_base::SocketAddress(host, port));
         }
       }
     }
   }
 
@@ skipping 61 matching lines @@
 void SessionStartRequest::OnOutgoingMessage(
     cricket::SessionManager* session_manager,
     const buzz::XmlElement* stanza) {
   // TODO(ajwong): Are we just supposed to not use |session_manager|?
   DCHECK_EQ(session_manager, session_manager_);
   scoped_ptr<buzz::XmlElement> stanza_copy(new buzz::XmlElement(*stanza));
   request_->SendRawIq(stanza_copy.get());
 }
 
 }  // namespace remoting