Follow @Openwall on Twitter for new release announcements and other news
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Tue, 24 Mar 2020 13:42:31 -0400
From: Rich Felker <dalias@...c.org>
To: musl@...ts.openwall.com
Subject: Re: [Bug] Do not ignore membarrier return code

On Tue, Mar 24, 2020 at 11:08:10AM -0500, A. Wilcox wrote:
> On 24/03/2020 09:01, Rich Felker wrote:
> > On Tue, Mar 24, 2020 at 02:53:21PM +0100, Julio Guerra wrote:
> >>> But I wanted to give you more details with stack traces of the
> >>> segfault by retrying today with gdb but I cannot reproduce it
> >>> anymore...!
> >>> I'll retry later to see if I see the error again...
> >>
> >> The node:10.19-alpine image has been updated since yesterday! That may
> >> be why I don't have the issue anymore? It updated the latest
> >> alpine:3.11 image, so 3.11.5.
> >> The changelog talks says "main/musl: backport fixes from 1.2.0":
> >> https://www.alpinelinux.org/posts/Alpine-3.11.5-released.html
> > 
> > I'm not aware of any changes that are at all related to your problem.
> > 
> > Rich
> 
> My first guess is something to do with the really egregious inspector
> signal stack bug.  Our patch for Node 10.x is at:
> 
> https://code.foxkit.us/adelie/packages/blob/master/user/node/stack-silliness.patch
> 
> Alpine doesn't patch it for Node 12 because Node 12 has this code to
> purportedly "handle musl":
> 
> https://github.com/nodejs/node/blob/master/src/inspector_agent.cc#L112
> 
> But I still feel like it may be wrong.

I think this may be the cause. I don't think Alpine ever had the patch
Adélie had to fix this bug in node, but 3.11.5 has Node 12.15.0 which
has the above fix. It looks like Alpine 3.11.0 has 12.14.0 which
should also have the fix though...

Rich

Powered by blists - more mailing lists

Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.