]> code.ossystems Code Review - openembedded-core.git/commitdiff
go-runtime: prevent host leakage into target objects
authorMatt Madison <matt@madison.systems>
Wed, 4 Oct 2017 18:21:52 +0000 (18:21 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 6 Oct 2017 11:03:32 +0000 (12:03 +0100)
When building for a target whose architecture matches
the build host's, the second pass through make.bash
to build the shareable runtime would also overwrite
the target's static cgo library with host-compatibile
binaries.

Fix this by running the host-side build once and
target-only passes of make.bash twice, for static
and shareable.  This ensures that what gets installed
is target-compatible.

Also fix an issue with x86-64 targets running MUSL by
removing the pre-built (for glibc) objects for the
race detector runtime before building.

[YOCTO #12136]

Signed-off-by: Matt Madison <matt@madison.systems>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/recipes-devtools/go/go-runtime.inc

index 934d1aaeb935bdc13559bda3864324841ac5dff8..f181dc7a29931e2dbcbb21eca1ff95dba8c15ec2 100644 (file)
@@ -15,7 +15,13 @@ export CC_FOR_TARGET="${CC}"
 export CXX_FOR_TARGET="${CXX}"
 export GOROOT_OVERRIDE = "1"
 
-do_configure[noexec] = "1"
+do_configure() {
+       :
+}
+
+do_configure_libc-musl() {
+       rm -f ${S}/src/runtime/race/*.syso
+}
 
 do_compile() {
        export GOBIN="${B}/bin"
@@ -23,9 +29,13 @@ do_compile() {
        rm -rf ${GOBIN} ${B}/pkg
        mkdir ${GOBIN}
        cd src
-       GO_FLAGS="" ./make.bash
+       ./make.bash --host-only
+       cp ${B}/pkg/tool/${BUILD_GOTUPLE}/go_bootstrap ${B}
+       rm -rf ${B}/pkg/${TARGET_GOTUPLE}
+       ./make.bash --target-only
        if [ -n "${GO_DYNLINK}" ]; then
-               GO_FLAGS="-buildmode=shared" GO_LDFLAGS="-extldflags \"${LDFLAGS}\"" ./make.bash
+               cp ${B}/go_bootstrap ${B}/pkg/tool/${BUILD_GOTUPLE}
+               GO_FLAGS="-buildmode=shared" GO_LDFLAGS="-extldflags \"${LDFLAGS}\"" ./make.bash --target-only
        fi
        cd ${B}
 }
@@ -41,8 +51,9 @@ do_install() {
        rm -rf ${D}${libdir}/go/pkg/obj
        rm -rf ${D}${libdir}/go/pkg/bootstrap
        find src -mindepth 1 -maxdepth 1 -type d | while read srcdir; do
-               [ "$srcdir" = "./cmd" ] || cp --preserve=mode,timestamps -R $srcdir ${D}${libdir}/go/src/
+               cp --preserve=mode,timestamps -R $srcdir ${D}${libdir}/go/src/
        done
+       rm -f ${D}${libdir}/go/src/cmd/dist/dist
 }
 
 # Remove test binaries that cannot be relocated