On 09/20/2011 03:44 PM, Alon Levy wrote:
On Mon, Sep 19, 2011 at 12:47:07PM +0300, Yonit Halperin wrote:
RHBZ 725009, 738270

Signed-off-by: Yonit Halperin<[email protected]>
---
  client/red_client.cpp |   19 ++++++++++++++++---
  1 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/client/red_client.cpp b/client/red_client.cpp
index 3de359d..a7f9cba 100644
--- a/client/red_client.cpp
+++ b/client/red_client.cpp
@@ -257,9 +257,15 @@ void* Migrate::worker_main(void *data)

  void Migrate::start(const SpiceMsgMainMigrationBegin* migrate)
  {
+    std::string cert_subject;
+    uint32_t peer_major;
+    uint32_t peer_minor;
+
      DBG(0, "");
      abort();

So this is removed in a later patch? if you do a respin can you move the 
removal of
the abort to this patch?

This abort is not removed. It is the abort of the Migrate class, it just aborts a previous migration if started (disconnects the channels from the target)
-    if ((_client.get_peer_major() == 1)&&  (_client.get_peer_minor()<  1)) {
+    peer_major = _client.get_peer_major();
+    peer_minor = _client.get_peer_minor();
+    if ((peer_major == 1)&&  (peer_minor<  1)) {
          LOG_INFO("server minor version incompatible for destination 
authentication"
                   "(missing dest pubkey in SpiceMsgMainMigrationBegin)");
          OldRedMigrationBegin* old_migrate = (OldRedMigrationBegin*)migrate;
@@ -271,8 +277,15 @@ void Migrate::start(const SpiceMsgMainMigrationBegin* 
migrate)
          _host.assign((char *)migrate->host_data);
          _port = migrate->port ? migrate->port : -1;
          _sport = migrate->sport ? migrate->sport : -1;
-        _auth_options.type_flags = 
RedPeer::HostAuthOptions::HOST_AUTH_OP_PUBKEY;
-        _auth_options.host_pubkey.assign(migrate->pub_key_data, 
migrate->pub_key_data + migrate->pub_key_size);
+        if ((peer_major == 1) || (peer_major == 2&&  peer_minor<  1)) {
+            _auth_options.type_flags = 
RedPeer::HostAuthOptions::HOST_AUTH_OP_PUBKEY;
+            _auth_options.host_pubkey.assign(migrate->pub_key_data, 
migrate->pub_key_data +
+                                             migrate->pub_key_size);
+        } else {
+            if (migrate->cert_subject_size != 0) {
+                _auth_options.set_cert_subject((char 
*)migrate->cert_subject_data);
+            }
+        }
      }

      _con_ciphers = _client.get_connection_ciphers();
--
1.7.4.4


_______________________________________________
Spice-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to