On Mon, 2011-04-11 at 12:42 -0400, Michael Santos wrote:
> ---
>  src/lxc/namespace.c |    7 ++++++-
>  1 files changed, 6 insertions(+), 1 deletions(-)
> 
> diff --git a/src/lxc/namespace.c b/src/lxc/namespace.c
> index 8263fe7..5928eaa 100644
> --- a/src/lxc/namespace.c
> +++ b/src/lxc/namespace.c
> @@ -67,9 +67,14 @@ pid_t lxc_clone(int (*fn)(void *), void *arg, int flags)
>       };
>  
>       long stack_size = sysconf(_SC_PAGESIZE);
> -     void *stack = alloca(stack_size) + stack_size;
> +     void *stack = alloca(stack_size);
>       pid_t ret;
>  
> +     if (stack == NULL)
> +             ERROR("failed to allocate memory");
> +
> +     stack += stack_size;
> +

Please see the man page, in particular "Notes on the GNU Version" and
"BUGS", for why this patch doesn't really make sense.

http://www.kernel.org/doc/man-pages/online/pages/man3/alloca.3.html



------------------------------------------------------------------------------
Xperia(TM) PLAY
It's a major breakthrough. An authentic gaming
smartphone on the nation's most reliable network.
And it wants your games.
http://p.sf.net/sfu/verizon-sfdev
_______________________________________________
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel

Reply via email to