Re: [PATCH] qemu: vnc: error out for invalid port number

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 13, 2015 at 09:51:08AM +0200, Martin Kletzander wrote:
> On Tue, May 12, 2015 at 07:00:07PM +0200, Pavel Hrdina wrote:
> >In the XML we have the vnc port number, but QEMU takes on command line
> >a vnc screen number, it's port-5900.  We should fail with error message
> >that only ports in range [5900,65535] are valid.
> >
> >Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1164966
> >
> >Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx>
> >---
> > src/qemu/qemu_command.c | 96 +++++++++++++++++++++++++++----------------------
> > 1 file changed, 53 insertions(+), 43 deletions(-)
> >
> 
> What a weird diff, have you used --patience for formatting?  It might
> be worth putting a note here that the diff is best viewed using '-w'
> or even better '-w --word-diff' so that reviewer notices immediatelly
> that most of the change is just whitespace movement.
> 
> ACK.

Thanks for review.  Yes, I've used --patience for formatting and 
I would mention the -w or --word-diff options, if I knew them.

I'll push it shortly.

Pavel

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]