Retrieving visuals offered by upstream Xserver is broken in several ways:
a) duplicate elimination breaks out too fast: when a duplicate is found,
it doesn't just skips that one, it completely breaks out the loop,
so subsequent upstream visuals aren't considered anymore. that's leading
to (unpredictable) limit on available color depths (depending on the order
reported by upstream sever)
b) buffer overflow when user specificed different depth/class than default one:
xnestOpenScreen() looks into the wrong table: it's local visuals[] array,
instead of the global (non-dedup'ed) list fetched by xlib. The visuals[]
array is *much* smaller (deduplicated) than the xnestVisuals[] array,
and xnestDefaultVisualIndex is likely to point outside of visual[]'s bounds.
To make it actually work against an Xorg upstream server, the upstream server
needs fix for another bug in the DIX:
https://gitlab.freedesktop.org/xorg/xserver/-/issues/1741https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1644
Closes: https://gitlab.freedesktop.org/xorg/xserver/-/issues/1742
Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
This is strictly the application of the script 'x-indent-all.sh'
from util/modular. Compared to the patch that Daniel posted in
January, I've added a few indent flags:
-bap
-psl
-T PrivatePtr
-T pmWait
-T _XFUNCPROTOBEGIN
-T _XFUNCPROTOEND
-T _X_EXPORT
The typedefs were needed to make the output of sdksyms.sh match the
previous output, otherwise, the code is formatted badly enough that
sdksyms.sh generates incorrect output.
The generated code was compared with the previous version and found to
be essentially identical -- "assert" line numbers and BUILD_TIME were
the only differences found.
The comparison was done with this script:
dir1=$1
dir2=$2
for dir in $dir1 $dir2; do
(cd $dir && find . -name '*.o' | while read file; do
dir=`dirname $file`
base=`basename $file .o`
dump=$dir/$base.dump
objdump -d $file > $dump
done)
done
find $dir1 -name '*.dump' | while read dump; do
otherdump=`echo $dump | sed "s;$dir1;$dir2;"`
diff -u $dump $otherdump
done
Signed-off-by: Keith Packard <keithp@keithp.com>
Acked-by: Daniel Stone <daniel@fooishbar.org>
Acked-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Enrico Weigelt, metux IT consult
Keith Packard
Adam Jackson
Adam Jackson
Egbert Eich
Kaleb Keithley