[Spice-devel] [PATCH spice-gtk v3 1/6] display-gst: check codec type before creating decoder

Christophe de Dinechin cdupontd at redhat.com
Fri Jun 9 15:44:28 UTC 2017


> On 8 Jun 2017, at 16:39, Frediano Ziglio <fziglio at redhat.com> wrote:
> 
>> 
>> On Thu, Jun 08, 2017 at 03:55:11PM +0200, Victor Toso wrote:
>>>> 
>>>> Not really worth all that discussion, your patch is ok :)
>>> 
>>> Ah, I liked the discusison mostly because this would be a spice <->
>>> spice-gtk communication bug and I'm not sure either if we should go for
>>> CRITICAL or WARNING messages in such cases.
>>> 
>>> I mean, in the past, for spice server, there was discussion about when
>>> to assert(). The result was that only when it would be a bug inside the
>>> component (never on client's input/configuration for instance)
>>> 
>>> Anyway, thanks for the discussion :)
>> 
>> To tie this in with the logging discussion, you can read the latest
>> comment from ebassi in this blog post about g_log_structured ;)
>> 
>> https://blog.gtk.org/2017/05/04/logging-and-more/
>> 
>> Christophe
>> 
> 
> Wrong thread? Looks like the structured logging would be more
> appropriate in Chistophe D thread.
> 
> About CRITICAL or WARNING personally I think if you are detecting
> that server for some reason is sending garbage telling the used on
> the UI and closing the connection/program is a good solution for
> an user application.

Only a good solution after we have cleaned up the existing issues there. With spicy today, I have 5 CRITICAL and 5 WARNING before the client window opens. The program won’t be that useful if it does not even start.



> For the server is a completely different case as the termination
> cause a DoS of the entire VM and also we don't have a user to
> communicate to.
> 
> Frediano
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



More information about the Spice-devel mailing list