Re: [PATCH spice-gtk] coroutine: abort on OOM or impossible conditions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 19, 2013 at 05:30:55PM +0100, Marc-André Lureau wrote:
> @@ -75,21 +73,13 @@ int coroutine_init(struct coroutine *co)
>  			    PROT_READ | PROT_WRITE,
>  			    MAP_PRIVATE | MAP_ANONYMOUS,
>  			    -1, 0);
> -	if (co->cc.stack == MAP_FAILED)
> -		g_error("Failed to allocate %u bytes for coroutine stack: %s",
> -			(unsigned)co->stack_size, strerror(errno));
> +	g_assert(co->cc.stack != MAP_FAILED);
> +

The g_error() here is much better imo. Actually I'd favour more use of
g_error() and less use of g_assert throughout this patch.

Christophe

Attachment: pgpOtcP1EOFC1.pgp
Description: PGP signature

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]