rdesktop seamless integration crashes on 64bit

Bug #667275 reported by Stephan Diestelhorst on 2010-10-27
This bug affects 1 person
Affects Status Importance Assigned to Milestone
rdesktop (Ubuntu)
Kees Cook

Bug Description

Binary package hint: rdesktop

Using rdesktop on an AMD64 bit architecutre with the seamlessRDP feature
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

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!


Changed in rdesktop (Ubuntu):
assignee: nobody → Kees Cook (kees)
status: New → Fix Committed
importance: Undecided → Medium
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  Edit
Everyone can see this information.

Other bug subscribers