Re: [Spice-devel] [spice-devel] spice-xpi rhbz#805602

2012-04-03 Thread David Jaša
Marc-André Lureau píše v Út 03. 04. 2012 v 08:39 -0400: > > - Mensaje original - > > +if (port < 0 && sport < 0) > > +{ > > +if (port < 0) > > +LOG_ERROR("invalid port: " << m_port); > > +if (sport < 0) > > +LOG_ERROR("invalid secure port: "

Re: [Spice-devel] [spice-devel] spice-xpi rhbz#805602

2012-04-03 Thread Marc-André Lureau
- Mensaje original - > +if (port < 0 && sport < 0) > +{ > +if (port < 0) > +LOG_ERROR("invalid port: " << m_port); > +if (sport < 0) > +LOG_ERROR("invalid secure port: " << m_secure_port); > +return; > +} If one of the value is

Re: [Spice-devel] [spice-devel] spice-xpi rhbz#805602

2012-04-03 Thread Christophe Fergeau
On Tue, Apr 03, 2012 at 02:28:07PM +0200, Peter Hatina wrote: > Hi, > > can you do a review for me? > > From 18f6fcabe35be03fe88f1a08b79e22ef505bbacd Mon Sep 17 00:00:00 2001 > From: Peter Hatina > Date: Tue, 3 Apr 2012 13:35:55 +0200 > Subject: [PATCH] Validate port values > > --- > SpiceXPI/

[Spice-devel] [spice-devel] spice-xpi rhbz#805602

2012-04-03 Thread Peter Hatina
Hi, can you do a review for me? >From 18f6fcabe35be03fe88f1a08b79e22ef505bbacd Mon Sep 17 00:00:00 2001 From: Peter Hatina Date: Tue, 3 Apr 2012 13:35:55 +0200 Subject: [PATCH] Validate port values --- SpiceXPI/src/plugin/plugin.cpp | 29 +++-- 1 files changed, 27 ins