[ clisp-Bugs-3292605 ] new-clx get-property unpacks properties incorrectly on 64bit

[SourceForge.net <no...@so...>]

Bugs item #3292605, was opened at 2011-04-25 12:17
Message generated for change (Tracker Item Submitted) made by dangerousben
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101355&aid=3292605&group_id=1355

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: clx
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Ben Spencer (dangerousben)
Assigned to: Bruno Haible (haible)
Summary: new-clx get-property unpacks properties incorrectly on 64bit

Initial Comment:
>From the XGetWindowProperty man page: 'If the returned format is 32, the property data will be stored as an array of longs (which in a 64-bit application will be 64-bit values that are padded in the upper 4 bytes).'  But XLIB:GET-PROPERTY casts prop_return to a uint32 when actual_format is 64.  Attached patch casts it to long instead, which fixes the issue at least on my amd64 machine.

This was the ultimate cause of the issue described here (and explains why other clx implementations were apparently unaffected): http://sourceforge.net/mailarchive/forum.php?thread_name=4A954322.80106%40gnu.org&forum_name=clisp-devel

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=101355&aid=3292605&group_id=1355