diff options
author | Mihai Moldovan <ïonic@ionic.de> | 2016-05-27 15:52:50 +0000 |
---|---|---|
committer | Mihai Moldovan <ïonic@ionic.de> | 2016-06-13 15:06:36 +0000 |
commit | 639a2c36f3a52f206d9b408ffc4580edc05f1392 (patch) | |
tree | 8c3f381f1e6bd9a18a32e0410fc4b64cbfa571d1 /nx-X11/programs/Xserver/os/utils.c | |
parent | 35613d1dcaa47fbbc6aacc95fdda16b48dd8e740 (diff) | |
download | nx-libs-639a2c36f3a52f206d9b408ffc4580edc05f1392.tar.gz nx-libs-639a2c36f3a52f206d9b408ffc4580edc05f1392.tar.bz2 nx-libs-639a2c36f3a52f206d9b408ffc4580edc05f1392.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 'nx-X11/programs/Xserver/os/utils.c')
0 files changed, 0 insertions, 0 deletions