[PATCH:libXi] Remove fallback for _XEatDataWords, require libX11 1.6 for it
Peter Hutterer
peter.hutterer at who-t.net
Sun Nov 24 16:19:48 PST 2013
On Sat, Nov 23, 2013 at 09:51:57AM -0800, Alan Coopersmith wrote:
> From: Michael Joost <mehl at michael-joost.de>
>
> _XEatDataWords was orignally introduced with the May 2013 security
> patches, and in order to ease the process of delivering those,
> fallback versions of _XEatDataWords were included in the X extension
> library patches so they could be applied to older versions that didn't
> have libX11 1.6 yet. Now that we're past that hurdle, we can drop
> the fallbacks and just require libX11 1.6 for building new versions
> of the extension libraries.
>
> Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
thank you, pushed as c4b261d..288f336 master -> master
Cheers,
Peter
> ---
> configure.ac | 8 +-------
> src/XIint.h | 13 -------------
> 2 files changed, 1 insertion(+), 20 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index a66fcee..54ec255 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -27,16 +27,10 @@ XORG_WITH_ASCIIDOC(8.4.5)
> XORG_CHECK_MALLOC_ZERO
>
> # Obtain compiler/linker options for dependencies
> -PKG_CHECK_MODULES(XI, [xproto >= 7.0.13] [x11 >= 1.4.99.1] [xextproto >= 7.0.3] [xext >= 1.0.99.1] [inputproto >= 2.2.99.1])
> +PKG_CHECK_MODULES(XI, [xproto >= 7.0.13] [x11 >= 1.6] [xextproto >= 7.0.3] [xext >= 1.0.99.1] [inputproto >= 2.2.99.1])
> # CFLAGS only for PointerBarrier typedef
> PKG_CHECK_MODULES(XFIXES, [xfixes >= 5])
>
> -# Check for _XEatDataWords function that may be patched into older Xlib releases
> -SAVE_LIBS="$LIBS"
> -LIBS="$XI_LIBS"
> -AC_CHECK_FUNCS([_XEatDataWords])
> -LIBS="$SAVE_LIBS"
> -
> # Check for xmlto and asciidoc for man page conversion
> # (only needed by people building tarballs)
> if test "$have_xmlto" = yes && test "$have_asciidoc" = yes; then
> diff --git a/src/XIint.h b/src/XIint.h
> index 3ddc3c5..99f3652 100644
> --- a/src/XIint.h
> +++ b/src/XIint.h
> @@ -83,18 +83,5 @@ next_block(void **ptr, int size) {
> return ret;
> }
>
> -#ifndef HAVE__XEATDATAWORDS
> -#include <X11/Xmd.h> /* for LONG64 on 64-bit platforms */
> -#include <limits.h>
> -
> -static inline void _XEatDataWords(Display *dpy, unsigned long n)
> -{
> -# ifndef LONG64
> - if (n >= (ULONG_MAX >> 2))
> - _XIOError(dpy);
> -# endif
> - _XEatData (dpy, n << 2);
> -}
> -#endif
>
> #endif
> --
> 1.7.9.2
>
More information about the xorg-devel
mailing list