diff options
| author | Yonit Halperin <yhalperi@redhat.com> | 2012-05-24 11:00:33 +0300 |
|---|---|---|
| committer | Yonit Halperin <yhalperi@redhat.com> | 2012-05-24 11:00:33 +0300 |
| commit | c59b2884a2f7fc953fdb263085830b65e8bdcaef (patch) | |
| tree | 78c6302afb2c6cd9ae111ccb113583a691a2d159 /server/red_worker.c | |
| parent | 038ae54b1c4c43a63d346bdfe2db52b48f20a22e (diff) | |
| download | spice-c59b2884a2f7fc953fdb263085830b65e8bdcaef.tar.gz spice-c59b2884a2f7fc953fdb263085830b65e8bdcaef.tar.xz spice-c59b2884a2f7fc953fdb263085830b65e8bdcaef.zip | |
server/red_channel: remove red_channel_client_item_being_sent
The above routine was risky, since red_channel_client_init_send_data
can also be called with item==NULL. Thus, not all pipe items can be tracked.
The one call that was made for this routine was not necessary.
Diffstat (limited to 'server/red_worker.c')
| -rw-r--r-- | server/red_worker.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/server/red_worker.c b/server/red_worker.c index 9e5624ba..2709486b 100644 --- a/server/red_worker.c +++ b/server/red_worker.c @@ -10364,9 +10364,7 @@ static void red_wait_pipe_item_sent(RedChannelClient *rcc, PipeItem *item) spice_printerr("timeout"); red_channel_client_disconnect(rcc); } else { - if (red_channel_client_item_being_sent(rcc, item)) { - red_wait_outgoing_item(rcc); - } + red_wait_outgoing_item(rcc); } channel->channel_cbs.release_item(rcc, item, FALSE); } |