From: Kurt Pattyn <pattyn.kurt@gmail.com>
Date: Sun, 16 Feb 2014 11:24:41 +0000 (+0100)
Subject: Fix non-strict warning from AutoBahn
X-Git-Tag: v5.3.0-alpha1~4
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8cf5cb96a8f49b4a2fd816e10dab385463becb43;p=contrib%2Fqtwebsockets.git

Fix non-strict warning from AutoBahn

When the reserved bits were set to 3, then the connection was aborted.
No a close message is sent before the connection is closed.

Change-Id: Iab365a4720c6588e5c6ac73813f63d26c3312265
Reviewed-by: Kurt Pattyn <pattyn.kurt@gmail.com>
---

diff --git a/src/websockets/qwebsocketframe.cpp b/src/websockets/qwebsocketframe.cpp
index 7aaa6aa..431710c 100644
--- a/src/websockets/qwebsocketframe.cpp
+++ b/src/websockets/qwebsocketframe.cpp
@@ -536,8 +536,6 @@ void QWebSocketFrame::setError(QWebSocketProtocol::CloseCode code, const QString
  */
 bool QWebSocketFrame::checkValidity()
 {
-    if (isValid())
-        return true;
     if (Q_UNLIKELY(m_rsv1 || m_rsv2 || m_rsv3)) {
         setError(QWebSocketProtocol::CloseCodeProtocolError, QObject::tr("Rsv field is non-zero"));
     } else if (Q_UNLIKELY(QWebSocketProtocol::isOpCodeReserved(m_opCode))) {