[Spice-devel,spice-gtk,v2,1/8] channel-base: remove bad check of SpiceMsgWaitForChannels validity

Submitted by Yonit Halperin on Aug. 25, 2012, 8:20 p.m.

Details

Message ID 1345926024-18138-2-git-send-email-yhalperi@redhat.com
State New
Headers show

Not browsing as part of any series.

Commit Message

Yonit Halperin Aug. 25, 2012, 8:20 p.m.
SpiceMsgWaitForChannels is not packed. Comparing the original
msg size to SpiceMsgWaitForChannels is wrong.
---
 gtk/channel-base.c |    3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

Patch hide | download patch | download mbox

diff --git a/gtk/channel-base.c b/gtk/channel-base.c
index cc4d242..2968f42 100644
--- a/gtk/channel-base.c
+++ b/gtk/channel-base.c
@@ -120,9 +120,6 @@  void spice_channel_handle_wait_for_channels(SpiceChannel *channel, SpiceMsgIn *i
     SpiceMsgWaitForChannels *wfc = spice_msg_in_parsed(in);
     int i;
 
-    g_return_if_fail(spice_header_get_msg_size(in->header, channel->priv->use_mini_header) >=
-                     sizeof(*wfc) + wfc->wait_count * sizeof(wfc->wait_list[0]));
-
     for (i = 0; i < wfc->wait_count; ++i) {
         WaitForChannelData data = {
             .wait = wfc->wait_list + i,