[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