HiEl mié, 19-09-2018 a las 15:30 +0200, Victor Toso escribió:Hi,On Mon, Aug 13, 2018 at 01:44:37PM +0200, Javier Celaya wrote:Hi, sorry for the late response, I was on holidaysEl mar, 24-07-2018 a las 17:58 +0200, Marc-André Lureau escribió:On Mon, Jul 16, 2018 at 6:45 PM, Javier Celaya<javier.celaya@flexvdi.com> wrote:The main channel reference d->main of SpiceWidget isinitialized onconstruction. However, it is used a couple of timesbefore that momentwhile setting the default values of someproperties, like resize-guest.This results in some annoying criticallog messages. This commit makessure that d->main is not used in suchsituations if it is NULL.Could you provide a reproducer of such warnings? I don't rememberseeing one.I see them each time I connect to a Spice session with spicy. Followme:- These warnings appear because d->main is NULL.- d->main is set in spice-widget.c:channel_new function, whenthe new channel is the main channel.- channel_new is called from spice_display_constructed with thelist of channels that already exist; obviously, the mainchannel is one of them.- There are SpiceDisplay properties that are set duringconstruction, before spice_display_constructed is called, thattrigger the code I am fixing with this patch.Makes sense.For instance, after setting "resize-guest" to its default valueTRUE, update_ready is called.So I do not see how you are notseeing these warnings. It does not seem to be a race condition,it happens always.resize-guest-enable's default is FALSE sospice_main_channel_update_display_enabled() would not be calledand not sure how you are manage to trigger it?My bad. We use a slightly modified version of spice-gtk for the flexVDI clients, and I did not remember that one of the changes is that resize-guest is set to TRUE by default. Maybe that's the reason we see the warning and you don't.
I've added a g_warn_if_fail(d->main != NULL); and indeed thed->main can be null at that point, so I'm keen to take the patchalthough a different solution might be better...Could you send a v2 with d->main check against NULL as Fredianomentioned and add the critical you see on commit log forcompleteness ?Sorry for delay, so many patches!VictorNever mind! Patch is on its way--- src/spice-widget.c | 4 ++-- 1 file changed, 2 insertions(+), 2deletions(-)diff --git a/src/spice-widget.c b/src/spice-widget.cindex72f5334..5187b0c 100644--- a/src/spice-widget.c+++ b/src/spice-widget.c@@ -255,7 +255,7 @@ static void update_ready(SpiceDisplay*display) * state here. If 'resize-guest' is false, we canassume that the * application will manage the state of thedisplays. */- if (d->resize_guest_enable) {+ if (d-resize_guest_enable && d->main){ spice_main_channel_update_display_enabled(d->main,get_display_id(display), ready, TRUE); }@@ -1247,7 +1247,7 @@ static void recalc_geometry(GtkWidget*widget) d->area.width, d-area.height, d->ww, d->wh, zoom);- if (d->resize_guest_enable)+ if (d->resize_guest_enable && d-main) spice_main_channel_update_display(d->main,get_display_id(display), d-area.x, d->area.y, d->ww / zoom, d->wh / zoom, TRUE);It looks like this may not give you the desired display size.Again, the warning happens when the property "scaling" is set duringSpiceDisplay construction, which calls scaling_update. You cannot callspice_main_channel_update_display with d->main == NULL, so this checkis harmless.}--2.17.0_______________________________________________Spice-devel mailinglistSpice-devel@lists.freedesktop.orghttps://lists.freedesktop.org/mailman/listinfo/spice-devel--Javier Celaya AlastruéChief Technology Officerjavier.celaya@xxxxxxxxxxx+34696969959j_celayaLegal Information and Privacy PolicyPolítica de confidencialidadEste mensaje y los ficheros anexos son confidenciales dirigiéndose exclusivamente al destinatario mencionado en el encabezamiento. Si usted ha recibido este correo por error, tenga la amabilidad de eliminarlo de su sistema y no divulgar el contenido a terceros. Los datos personales facilitados por usted o por terceros serán tratados por FLEXIBLE SOFTWARE SOLUTIONS S.L.U. con la finalidad de gestionar y mantener los contactos y relaciones que se produzcan como consecuencia de la relación que mantiene con FLEXIBLE SOFTWARE SOLUTIONS S.L.U. Normalmente, la base jurídica que legitima este tratamiento, será su consentimiento, el interés legítimo o la necesidad para gestionar una relación contractual o similar. El plazo de conservación de sus datos vendrá determinado por la relación que mantiene con nosotros. Para más información al respecto, o para ejercer sus derechos de acceso, rectificación, supresión, oposición, limitación o portabilidad, dirija una comunicación por escrito a FLEXIBLE SOFTWARE SOLUTIONS S.L.U: Avenida de Ranillas 1D, Planta 3, Oficina 3G, Zaragoza o al correo electrónico pdo@xxxxxxxxxxx. En caso de considerar vulnerado su derecho a la protección de datos personales, podrá interponer una reclamación ante la Agencia Española de Protección de Datos (www.agpd.es)._______________________________________________Spice-devel mailing listSpice-devel@xxxxxxxxxxxxxxxxxxxxxhttps://lists.freedesktop.org/mailman/listinfo/spice-devel--
![flexVDI]()
Javier Celaya Alastrué
Chief Technology Officer
+34696969959
j_celaya
Política de confidencialidad
Este mensaje y los ficheros anexos son confidenciales dirigiéndose exclusivamente al destinatario mencionado en el encabezamiento. Si usted ha recibido este correo por error, tenga la amabilidad de eliminarlo de su sistema y no divulgar el contenido a terceros. Los datos personales facilitados por usted o por terceros serán tratados por FLEXIBLE SOFTWARE SOLUTIONS S.L.U. con la finalidad de gestionar y mantener los contactos y relaciones que se produzcan como consecuencia de la relación que mantiene con FLEXIBLE SOFTWARE SOLUTIONS S.L.U. Normalmente, la base jurídica que legitima este tratamiento, será su consentimiento, el interés legítimo o la necesidad para gestionar una relación contractual o similar. El plazo de conservación de sus datos vendrá determinado por la relación que mantiene con nosotros. Para más información al respecto, o para ejercer sus derechos de acceso, rectificación, supresión, oposición, limitación o portabilidad, dirija una comunicación por escrito a FLEXIBLE SOFTWARE SOLUTIONS S.L.U: Avenida de Ranillas 1D, Planta 3, Oficina 3G, Zaragoza o al correo electrónico pdo@xxxxxxxxxxx. En caso de considerar vulnerado su derecho a la protección de datos personales, podrá interponer una reclamación ante la Agencia Española de Protección de Datos (www.agpd.es).
--
|
Javier Celaya Alastrué Chief Technology Officer |
|
|
Política de confidencialidad Este mensaje y los ficheros anexos son confidenciales dirigiéndose exclusivamente al destinatario mencionado en el encabezamiento. Si usted ha recibido este correo por error, tenga la amabilidad de eliminarlo de su sistema y no divulgar el contenido a terceros. Los datos personales facilitados por usted o por terceros serán tratados por FLEXIBLE SOFTWARE SOLUTIONS S.L.U. con la finalidad de gestionar y mantener los contactos y relaciones que se produzcan como consecuencia de la relación que mantiene con FLEXIBLE SOFTWARE SOLUTIONS S.L.U. Normalmente, la base jurídica que legitima este tratamiento, será su consentimiento, el interés legítimo o la necesidad para gestionar una relación contractual o similar. El plazo de conservación de sus datos vendrá determinado por la relación que mantiene con nosotros. Para más información al respecto, o para ejercer sus derechos de acceso, rectificación, supresión, oposición, limitación o portabilidad, dirija una comunicación por escrito a FLEXIBLE SOFTWARE SOLUTIONS S.L.U: Avenida de Ranillas 1D, Planta 3, Oficina 3G, Zaragoza o al correo electrónico pdo@xxxxxxxxxxx. En caso de considerar vulnerado su derecho a la protección de datos personales, podrá interponer una reclamación ante la Agencia Española de Protección de Datos (www.agpd.es). |
|
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel