[ccan] [PATCH 0/4] altstack: Small bugfixes

David Gibson david at gibson.dropbear.id.au
Tue Feb 16 17:09:21 AEDT 2016


So, by building myself a docker container with Ubuntu 12.04 (the
default distro used by Travis builds), I thought I'd figured out why
altstack has been breaking Travis builds for ages.

Unfortunately, I was wrong - 1/4 certainly fixes *a* bug on Ubuntu
12.04, but there's at least one more - I'm getting a SIGBUS in dn()
for reasons I haven't figured out.  I think it must be a different
behaviour in whatever kernel the Travis builders are using (so it
doesn't affect my local container build), but I don't know for sure.

Still might as well fix the bug I did track down.  While I was at it I
found a couple more bugs which I haven't seen bite in practice, but
I'm pretty sure could cause problems at least theoretically, hence
patches 2..3/4.

Finally 4/4 isn't a bugfix, but reworks some of the test code to make
failures a bit easier to diagnose.

David Gibson (4):
  altstack: Increase signal stack size
  altstack: Include config.h in run.c
  altstack: Declare memory clobbers
  altstack: Clarify checking macros

 ccan/altstack/altstack.c |  7 ++++---
 ccan/altstack/test/run.c | 30 +++++++++++++++++++++++++-----
 2 files changed, 29 insertions(+), 8 deletions(-)

-- 
2.5.0



More information about the ccan mailing list