Changeset 6349 in vbox for trunk/src/VBox/Frontends/VirtualBox

Timestamp:

Jan 14, 2008 12:42:42 PM (17 years ago)

Author:

vboxsync

Message:

2332: Registration feature:

1. Check for the handshake (auth) key correctness added.
2. Restriction for the http-request length removed.

Location:

trunk/src/VBox/Frontends/VirtualBox

Files:

• src/HappyHttp.cpp (modified) (1 diff)
• ui/VBoxRegistrationDlg.ui.h (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/HappyHttp.cpp

@@ -399 +399 @@
     m_State = REQ_STARTED;
 
-    char req[ 512 ];
-    sprintf( req, "%s %s HTTP/1.1", method, url );
-    m_Buffer.push_back( req );
+    m_Buffer.push_back (string (method) + " " + string (url) + " HTTP/1.1");
 
     putheader( "Host", m_Host.c_str() );    // required for HTTP1.1

trunk/src/VBox/Frontends/VirtualBox/ui/VBoxRegistrationDlg.ui.h

@@ -361 +361 @@
     if (mHandshake)
     {
+        /* Verifying key correctness */
+        if (QString (aTotalData).find (QRegExp ("^[a-zA-Z0-9]{32}$")))
+        {
+            abortRegisterRequest (tr ("Could not perform connection handshake."));
+            return;
+        }
+
         /* Registration arguments initializing */
         QString version = vboxGlobal().virtualBox().GetVersion();