[PATCH:xf86-video-modesetting] Include <alloca.h> on platforms that need it for alloca

Jeremy Huddleston jeremyhu at apple.com
Fri Apr 13 11:22:36 PDT 2012


Can we please instead fix this by removing the alloca() calls?

--Jeremy

"""
BUGS
     alloca() is machine and compiler dependent; its use is discouraged.

     alloca() is slightly unsafe because it cannot ensure that the pointer returned points to a valid and usable
     block of memory.  The allocation made may exceed the bounds of the stack, or even go further into other
     objects in memory, and alloca() cannot determine such an error.  Avoid alloca() with large unbounded allo-
     cations.

     The use of C99 variable-length arrays and alloca() in the same function will cause the lifetime of alloca's
     storage to be limited to the block containing the alloca()
"""

On Apr 6, 2012, at 9:33 PM, Alan Coopersmith <alan.coopersmith at oracle.com> wrote:

> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
> configure.ac |    1 +
> src/driver.c |    3 +++
> 2 files changed, 4 insertions(+)
> 
> diff --git a/configure.ac b/configure.ac
> index aa4ae9f..ddec78e 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -78,6 +78,7 @@ AM_CONDITIONAL(HAVE_XEXTPROTO_71, [ test "$HAVE_XEXTPROTO_71" = "yes" ])
> 
> # Checks for header files.
> AC_HEADER_STDC
> +AC_ALLOCA
> 
> PKG_CHECK_MODULES(DRM, [libdrm >= 2.2])
> PKG_CHECK_MODULES([PCIACCESS], [pciaccess >= 0.10])
> diff --git a/src/driver.c b/src/driver.c
> index 87431ff..514c423 100644
> --- a/src/driver.c
> +++ b/src/driver.c
> @@ -33,6 +33,9 @@
> #include "config.h"
> #endif
> 
> +#ifdef HAVE_ALLOCA_H
> +# include <alloca.h>
> +#endif
> #include <unistd.h>
> #include <fcntl.h>
> #include "xf86.h"
> -- 
> 1.7.9.2
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 



More information about the xorg-devel mailing list