[Spice-devel] [PATCH spice-gtk] xfer: send data message of size 0 for 0-size file

[Marc-André Lureau]

On Fri, Aug 29, 2014 at 9:55 AM, Christophe Fergeau <cfergeau at redhat.com>
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
> > Spice-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/spice-devel
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>
>


-- 
Marc-André Lureau
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20140829/92e11944/attachment.html>