Received: from vps892.directvps.nl (ikke.info [178.21.113.177])
	by nld3-dev1.alpinelinux.org (Postfix) with ESMTPS id 4A9A7781A6C
	for <alpine-aports@lists.alpinelinux.org>; Fri, 22 Nov 2019 11:39:35 +0000 (UTC)
Received: by vps892.directvps.nl (Postfix, from userid 1008)
	id 249AF4400E7; Fri, 22 Nov 2019 12:39:35 +0100 (CET)
Date: Fri, 22 Nov 2019 12:39:35 +0100
From: Kevin Daudt <kdaudt@alpinelinux.org>
To: alpine-mips-patches <info@mobile-stream.com>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [PATCH] main/yasm: fix random segfault in testsuite
Message-ID: <20191122113935.GI456781@alpha>
References: <20191115051655.D1ED1509EB@mx12.valuehost.ru>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20191115051655.D1ED1509EB@mx12.valuehost.ru>
User-Agent: Mutt/1.12.2 (2019-09-21)

On Fri, Nov 15, 2019 at 05:01:41AM +0000, alpine-mips-patches wrote:
> observable on mipseln8hf; the patch is from upstream.
> ---
>  ...84586228af6c362f970c84fce58da3fcbdec8.patch | 18 ++++++++++++++++++
>  main/yasm/APKBUILD                             |  8 +++++---
>  2 files changed, 23 insertions(+), 3 deletions(-)
>  create mode 100644 main/yasm/74184586228af6c362f970c84fce58da3fcbdec8.patch
> 
> diff --git a/main/yasm/74184586228af6c362f970c84fce58da3fcbdec8.patch b/main/yasm/74184586228af6c362f970c84fce58da3fcbdec8.patch
> new file mode 100644
> index 0000000000..3ec24e6f68
> --- /dev/null
> +++ b/main/yasm/74184586228af6c362f970c84fce58da3fcbdec8.patch
> @@ -0,0 +1,18 @@
> +commit 74184586228af6c362f970c84fce58da3fcbdec8
> +Author: Matt Oliver <protogonoi@gmail.com>
> +Date:   Tue Apr 4 00:34:47 2017 +1000
> +
> +    fix memory errors when using gas processor with coff object output.
> +
> +diff --git a/modules/objfmts/coff/coff-objfmt.c b/modules/objfmts/coff/coff-objfmt.c
> +index 6fcb0e3c..8f753bfe 100644
> +--- a/modules/objfmts/coff/coff-objfmt.c
> ++++ b/modules/objfmts/coff/coff-objfmt.c
> +@@ -290,6 +290,7 @@ coff_common_create(yasm_object *object)
> +     objfmt_coff->done_prolog = 0;
> +     objfmt_coff->unwind = NULL;
> +     objfmt_coff->ssym_imagebase = NULL;
> ++    objfmt_coff->def_sym = NULL;
> + 
> +     return objfmt_coff;
> + }
> diff --git a/main/yasm/APKBUILD b/main/yasm/APKBUILD
> index ea8eaff9da..94646c3558 100644
> --- a/main/yasm/APKBUILD
> +++ b/main/yasm/APKBUILD
> @@ -1,13 +1,14 @@
>  # Maintainer: Natanael Copa <ncopa@alpinelinux.org>
>  pkgname=yasm
>  pkgver=1.3.0
> -pkgrel=1
> +pkgrel=2
>  pkgdesc="A rewrite of NASM to allow for multiple syntax supported (NASM, TASM, GAS, etc.)"
>  url="http://www.tortall.net/projects/yasm/"
>  arch="all"
>  license="BSD"
>  subpackages="$pkgname-dev $pkgname-doc"
> -source="http://www.tortall.net/projects/$pkgname/releases/$pkgname-$pkgver.tar.gz"
> +source="http://www.tortall.net/projects/$pkgname/releases/$pkgname-$pkgver.tar.gz
> +	74184586228af6c362f970c84fce58da3fcbdec8.patch"
>  options="!checkroot"
>  
>  builddir="$srcdir/$pkgname-$pkgver"
> @@ -38,4 +39,5 @@ package() {
>  	install -Dm644 COPYING "$pkgdir"/usr/share/licenses/$pkgname/COPYING
>  }
>  
> -sha512sums="572d3b45568b10f58e48f1188c2d6bcbdd16429c8afaccc8c6d37859b45635e106885d679e41d0bee78c23822108c7ae75aa7475eed5ba58057e0a6fe1b68645  yasm-1.3.0.tar.gz"
> +sha512sums="572d3b45568b10f58e48f1188c2d6bcbdd16429c8afaccc8c6d37859b45635e106885d679e41d0bee78c23822108c7ae75aa7475eed5ba58057e0a6fe1b68645  yasm-1.3.0.tar.gz
> +e0220f7d29a83fc107dfb134dad29c3246fdaa2bcba4a1182fb93e9fd810c53cc97062771c26ee4a34d5e8dea1feb3094647bdc68b4d45734a062eb8d4afa3d1  74184586228af6c362f970c84fce58da3fcbdec8.patch"
> -- 
> 2.24.0
> 

This was applied as bc96fea (by maxice8).