[X2Go-Dev] Bug#900: Bug#900: Gedit, gnome-terminal and others crash in rootless mode

Mike Gabriel mike.gabriel at das-netzwerkteam.de
Tue Feb 23 15:29:08 CET 2016


Control: tags -1 patch
Control: forwarded -1 https://github.com/ArcticaProject/nx-libs/issues/82

Hi Camilo,

On  Do 02 Jul 2015 08:21:22 CEST, Camilo Alejandro Arboleda wrote:

Your bug report has just been moved [1] to the new upstream location  
of nx-libs on Github.

> Looking at the highlighted values, it seems that gedit is sending a
> malformed ChangeProperty request, and rootless is failing to process it.

Is it really a malformed request or a problem with broken BIG-REQUESTS  
support [2] in libXcomp3 (aka nxcomp)?

> Specifically the segment between lines 735-780, tries to set a property
> that is bigger than the maximum size required, but because it's a
> malformed request it ends up writing in memory outside the boundaries of
> the output buffer.
>
> Alternatives:
>
>  1. Ensure that nxagentExportProperty never writes beyond the boundaries
>     of the output buffer.
>  2. Resize the output buffer to match the required size
>     (ProcChangeProperty seems to do something similar).
>  3. Ignore big requests (see attached patch).

Is option 3. really the optimal approach? It feels like option 2.  
would be the way to go here...

Please continue, if possible for you, this discussion on Github.

Mike

[1] https://github.com/ArcticaProject/nx-libs/issues/82
[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=766299
-- 

DAS-NETZWERKTEAM
mike gabriel, herweg 7, 24357 fleckeby
fon: +49 (1520) 1976 148

GnuPG Key ID 0x25771B31
mail: mike.gabriel at das-netzwerkteam.de, http://das-netzwerkteam.de

freeBusy:
https://mail.das-netzwerkteam.de/mailxchange/kronolith/fb.php?u=m.gabriel%40das-netzwerkteam.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digitale PGP-Signatur
URL: <http://lists.x2go.org/pipermail/x2go-dev/attachments/20160223/f717db6b/attachment.pgp>


More information about the x2go-dev mailing list