[Top][All Lists]

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

Re: [Chicken-hackers] [PATCH] correct size calculation for ffi return ty

From: Peter Bex
Subject: Re: [Chicken-hackers] [PATCH] correct size calculation for ffi return types
Date: Sun, 6 Aug 2017 17:50:38 +0200
User-agent: NeoMutt/20170113 (1.7.2)

On Mon, Jul 31, 2017 at 06:31:28PM +0200, lemonboy wrote:
> Hello hackers,
> I managed to stumble across this nasty compiler bug while trying to figure out
> why some code was segfaulting in a FFI stub. It turns out that I forgot to 
> place
> a define-foreign-type form before the lines the type was referenced in and the
> compiler happily assumed it had size 0 and skipped the generation of the 
> scratch
> space buffer.
> Here's a simple patch, I'm not too fond of the catch-all case but it shouldn't
> be a problem (the `make check` shows no regression).

Like I said on IRC, I'm not quite sure what to do with this one.  A similar
fix is already part of CHICKEN 5 (5bc3b2d25768f9bd89bafa3b9dd4b85376b0c10a),

We could apply the error part to master too, but I don't know what kind
of breakage it will cause.  I know it's buggy and probably stupid and
unsafe to leave it, but I'm not sure it's worth risking potentially massive
breakage of existing code that already works.

What do the other hackers think?


Attachment: signature.asc
Description: PGP signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]