Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 580434

Summary: dev-lang/go-1.6.1 fails tests
Product: Gentoo Linux Reporter: Rick Farina (Zero_Chaos) <zerochaos>
Component: Current packagesAssignee: William Hubbs <williamh>
Status: RESOLVED OBSOLETE    
Severity: normal CC: hardened, kingjon3377
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: arm-test-failure build.log
amd64-test-failure build.log

Description Rick Farina (Zero_Chaos) gentoo-dev 2016-04-18 14:47:43 UTC
Created attachment 431130 [details]
arm-test-failure build.log

test failures on amd64 and arm hardened
Comment 1 Rick Farina (Zero_Chaos) gentoo-dev 2016-04-18 14:53:15 UTC
Created attachment 431132 [details]
amd64-test-failure build.log

amd64 build log
Comment 2 William Hubbs gentoo-dev 2016-04-23 23:37:11 UTC
These do not fail on the default profile, so I don't think this should
block stabilization.
Comment 3 Anthony Basile gentoo-dev 2016-04-23 23:54:53 UTC
(In reply to William Hubbs from comment #2)
> These do not fail on the default profile, so I don't think this should
> block stabilization.

The two failures are different.  I'm not sure what's causing the arm failure, but the amd64 failure is cause for concern.  The error

error while loading shared libraries: cannot make segment writable for relocation: Permission denied

indicates that the linker is trying to rewrite text for a relocation, aka a TEXTREL, which is bad for security and sharing of pages in memory.  TEXTRELs are usually caught by scanelf as part of our QA checks.  While these are revealed by the hardened toolchain, I don't think its the fault of the toolchain per se.  Does go work primarily with non-PIC code?

While I don't know if you want to make this a blocker to stabilization or now, we should understand what's going on here.
Comment 4 Anthony Basile gentoo-dev 2016-04-23 23:56:05 UTC
(In reply to Rick Farina (Zero_Chaos) from comment #1)
> Created attachment 431132 [details]
> amd64-test-failure build.log
> 
> amd64 build log

can I have emerge --info for this system.  also, can you look in dmesg's for any clues as to what's going on at the kernel level during the build.  i assume you're using a pax-hardened kernel.
Comment 5 William Hubbs gentoo-dev 2016-04-24 15:17:00 UTC
(In reply to Anthony Basile from comment #3)
> (In reply to William Hubbs from comment #2)
> > These do not fail on the default profile, so I don't think this should
> > block stabilization.
> 
> The two failures are different.  I'm not sure what's causing the arm
> failure, but the amd64 failure is cause for concern.  The error
> 
> error while loading shared libraries: cannot make segment writable for
> relocation: Permission denied
> 
> indicates that the linker is trying to rewrite text for a relocation, aka a
> TEXTREL, which is bad for security and sharing of pages in memory.  TEXTRELs
> are usually caught by scanelf as part of our QA checks.  While these are
> revealed by the hardened toolchain, I don't think its the fault of the
> toolchain per se.  Does go work primarily with non-PIC code?

I'm honestly not sure about that. what I would do if I were you is join
#go-nuts on freenode and see if you can get an answer from them.

> While I don't know if you want to make this a blocker to stabilization or
> now, we should understand what's going on here.

I agree that we want to know what is going on with this bug; I wasn't planning on closing it.

I definitely don't want to block stabilization, because there is another important vulnerability fixed in 1.6.1 which affects all earlier versions. That is covered in  bug #579314.
Comment 6 Anthony Basile gentoo-dev 2016-04-24 16:02:36 UTC
(In reply to William Hubbs from comment #5)
> I definitely don't want to block stabilization, because there is another
> important vulnerability fixed in 1.6.1 which affects all earlier versions.
> That is covered in  bug #579314.

yeah i saw that after i wrote this, so you're right, move ahead with stabilization.