Hi ----- Original Message ----- > On Tue, Sep 06, 2016 at 05:36:37PM +0400, Marc-André Lureau wrote: > > srcfifo && dstfifo must still be freed if socat is not present. > > > > Only unlink if the fifo was actually created. > > > > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> > > --- > > tests/test-io-channel-command.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > Signed-off-by: Daniel P. Berrange <berra...@redhat.com> > > I'll add this to my I/O queue unless you respin your current PULL request > with this included.
I don't know what's best. Peter, should I send a new pull request with the new patch? Or send a new one without it? Or you can pull the current request without this patch? thanks > > Regards, > Daniel > -- > |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- http://virt-manager.org :| > |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| >
