[Telepathy] [PATCH 1/3] fix checkValidProtocolName test in BaseConnection

Maksim Melnikau maxposedon at gmail.com
Sun Jan 6 11:12:10 PST 2013


BaseConnection::registerObject should retun false,
if checkValidProtocolName return false, e.g. if protocol name is not
valid

Signed-off-by: Maksim Melnikau <maxposedon at gmail.com>
---
 TelepathyQt/base-connection.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/TelepathyQt/base-connection.cpp b/TelepathyQt/base-connection.cpp
index 3897f07..e6bea0f 100644
--- a/TelepathyQt/base-connection.cpp
+++ b/TelepathyQt/base-connection.cpp
@@ -175,7 +175,7 @@ bool BaseConnection::registerObject(DBusError *error)
         return true;
     }
 
-    if (checkValidProtocolName(mPriv->protocolName)) {
+    if (!checkValidProtocolName(mPriv->protocolName)) {
         if (error) {
             error->set(TP_QT_ERROR_INVALID_ARGUMENT,
                 mPriv->protocolName + QLatin1String("is not a valid protocol name"));
-- 
1.8.1



More information about the telepathy mailing list