Created attachment 117184 [details]
off-by-one fixes ported to xorg git
I stumbled across an off-by-one fix for XKBMAlloc.c in NoMachine's NX code (http://code.x2go.org/gitweb?p=nx-libs.git;a=commitdiff;h=15cee47a496ef891923949ade073cf4ffabb9c73#patch3).
It turned out that this bug (along with some others) had been fixed in 2005 in the xfree86 code by Michal Maruška (see http://cgit.freedesktop.org/~libv/xfree86/commit/lib/X11?id=23202557b77fd2edd167ae00103dcd81782401d5)
(more Details here: https://email@example.com/msg07272.html)
These fixes never made their way to xorg, it seems.
At least the off-by-one fix should be included. I have attached a patch for this.
The XFree86 fixes were published under the new XFree86 license which X.Org
is unwilling to accept. If the original author was to agree to contribute
under the standard X11 license, then we could take them.
These fixes are quite trivial. So in case the original author is refusing or not answering (I have been waiting for 11 days now...) I don't see a good way to reimplement them in a different way. But the code is definitely buggy. So what do you suggest in that case?
Can you describe in words and in detail what the changes are?
Ie, such-and-such a variable in such-and-such a function has an off-by-one error (too large or too small) when such-and-such an operation is done on it, and the like.
That should make it possible for someone who hasn’t seen the xfree code to re-implement the fixes w/ a compatible license w/o infringing on the xfree license.
I have got an answer of the original author and he will provide a patch.
-- GitLab Migration Automatic Message --
This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.
You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/xorg/lib/libx11/issues/76.