On Fri, Aug 29, 2014 at 9:55 AM, Christophe Fergeau <[email protected]> wrote:
> Hey, > > On Thu, Aug 28, 2014 at 10:19:04PM +0200, Marc-André Lureau wrote: > > Make sure we send a xfer data message for 0-size files. > > This avoid leaking file descriptiors in guest agent when > > copying such files. > > Has this been tested or should I give it a try? ACK if this has been > tested. You could add a note that the bug was reported by Cody maybe > > Yes, I did testing. Ok, I'll add a Reported-by: Cody Chan <..> > > > > https://bugzilla.redhat.com/show_bug.cgi?id=1135099 > > --- > > gtk/channel-main.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/gtk/channel-main.c b/gtk/channel-main.c > > index f33b0fd..46c81c1 100644 > > --- a/gtk/channel-main.c > > +++ b/gtk/channel-main.c > > @@ -1637,9 +1637,11 @@ static void file_xfer_read_cb(GObject > *source_object, > > return; > > } > > > > - if (count > 0) { > > + if (count > 0 || task->file_size == 0) { > > task->read_bytes += count; > > file_xfer_queue(task, count); > > + if (count == 0) > > + return; > > file_xfer_flush_async(channel, task->cancellable, > > file_xfer_data_flushed_cb, task); > > task->pending = TRUE; > > -- > > 1.9.3 > > > > _______________________________________________ > > Spice-devel mailing list > > [email protected] > > http://lists.freedesktop.org/mailman/listinfo/spice-devel > > _______________________________________________ > Spice-devel mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/spice-devel > > -- Marc-André Lureau
_______________________________________________ Spice-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/spice-devel
