[Spice-devel] [PATCH spice-common 1/9] messages: Remove fields not used by the protocol

Frediano Ziglio fziglio at redhat.com
Sun Mar 3 19:10:22 UTC 2019


These fields are not used by the protocol.
Avoid spice-gtk and spice-server to use them by mistake.
This can cause memory errors (data_size is not used or
is not set correctly) and useless code (spice-gtk uses
the pub_key* fields but these fields are not sent to
the server as the protocol does not have them).

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 common/messages.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/common/messages.h b/common/messages.h
index 43d3602..f740a8c 100644
--- a/common/messages.h
+++ b/common/messages.h
@@ -43,7 +43,6 @@
 SPICE_BEGIN_DECLS
 
 typedef struct SpiceMsgData {
-    uint32_t data_size;
     uint8_t data[0];
 } SpiceMsgData;
 
@@ -75,9 +74,6 @@ typedef struct SpiceMigrationDstInfo {
     uint16_t sport;
     uint32_t host_size;
     uint8_t *host_data;
-    uint16_t pub_key_type;
-    uint32_t pub_key_size;
-    uint8_t *pub_key_data;
     uint32_t cert_subject_size;
     uint8_t *cert_subject_data;
 } SpiceMigrationDstInfo;
-- 
2.20.1



More information about the Spice-devel mailing list