diff options
author | Mihai Moldovan <ïonic@ionic.de> | 2016-05-27 15:52:50 +0000 |
---|---|---|
committer | Mike Gabriel <mike.gabriel@das-netzwerkteam.de> | 2016-06-21 04:06:27 +0200 |
commit | 129cc6afd4ac49b458659ae8bb0d5201942c2aab (patch) | |
tree | 8c3f381f1e6bd9a18a32e0410fc4b64cbfa571d1 /nxcomp | |
parent | f5eff9bc0a1134a16b1a6e67bfe3d0b4d7cd49e9 (diff) | |
download | nx-libs-129cc6afd4ac49b458659ae8bb0d5201942c2aab.tar.gz nx-libs-129cc6afd4ac49b458659ae8bb0d5201942c2aab.tar.bz2 nx-libs-129cc6afd4ac49b458659ae8bb0d5201942c2aab.zip |
nx-X11/programs/Xserver/hw/nxagent/Handlers.h: fix stack smashing related to different data type sizes between Xserver and Xlib.
While on the X server side we were expecting a 32bit value, the Xlib side in
nxcompext wrote a 64bit integer to the 32bit location, hence overwriting
"random" data on the stack (most notably the return adress.)
We can work around this by using the Xlib-based Window data type on the
server-side as well, but this problem is likely a more general one.
Calling functions from libcompext in nxagent sounds like a bad idea. Other code
locations might well be affected by issues alike to that as well.
In order to silence compiler warnings, use a "macro hack" for overriding the
"Window" type in NXlib.h.
Diffstat (limited to 'nxcomp')
0 files changed, 0 insertions, 0 deletions