rdesktop seamless integration crashes on 64bit

Bug #667275 reported by Stephan Diestelhorst
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
rdesktop (Ubuntu)
Fix Released
Medium
Kees Cook

Bug Description

Binary package hint: rdesktop

Using rdesktop on an AMD64 bit architecutre with the seamlessRDP feature
http://www.cendio.com/seamlessrdp/
crashes rdesktop whenever a new window is opened.

I've traces this back to some odd definition in the Xorg spec, where 32 bit values
are specified as long, regardless of architecture, and hence need to be 64 bit on
AMD64. The original code uses uint32_t, which causes crashes during parsing of
icons for an enhanced window manager hint.

The attached patch fixes the issue, and adds sanity checks to catch similar errors
during pointer arithmetic on icon data. The upstream ML seems to be dead, hence
the repost here, with the hope that Ubuntu integrates the patch.

The patch has been in heavy usage on my machine, and no furhter crashes occured.

Related branches

Revision history for this message
Stephan Diestelhorst (syon) wrote :
Revision history for this message
Kees Cook (kees) wrote :

Thanks for this patch! It looks like upstream has already fixed this in Subversion but not yet released an updated package. I'll pull the upstream patch (which includes another fix as well) and upload it. Thanks!

http://rdesktop.svn.sourceforge.net/viewvc/rdesktop/rdesktop/trunk/ewmhints.c?r1=1550&r2=1554&view=patch

Changed in rdesktop (Ubuntu):
assignee: nobody → Kees Cook (kees)
status: New → Fix Committed
importance: Undecided → Medium
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package rdesktop - 1.6.0-3ubuntu4

---------------
rdesktop (1.6.0-3ubuntu4) natty; urgency=low

  * debian/patches/debian/patches/16_lp667275.dpatch: fix more 64bit
    crashes, from upstream, thanks to Stephan Diestelhorst (LP: #667275).
 -- Kees Cook <email address hidden> Thu, 10 Mar 2011 15:55:09 -0800

Changed in rdesktop (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.