From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: pcre 8.37 with latest patches won't build
Date: Wed, 19 Aug 2015 20:34:21 +0100
Message-ID: <1440012861.2322.1.camel@ipfire.org>
In-Reply-To: <55D4CF3C.8080305@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============6729037269499585009=="
List-Id: <development.lists.ipfire.org>

--===============6729037269499585009==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Hi,

sorry, this is not your fault. I fucked that up.

I was sure that these patches built but clearly I did something wrong.

I reverted the commit so that you can pull and resume your build and
will then upload a new commit after I fixed this.

Thanks for the notice.

Best,
-Michael

On Wed, 2015-08-19 at 20:47 +0200, Matthias Fischer wrote:
> Hi,
> 
> yesterday I tried to build the latest 'next', but it failed. It
> couldn't 
> apply 'pcre-8.37-Fix-another-buffer-overflow.patch':
> 
> ***SNIP***
> Aug 18 19:03:54: Building pcre pcre-8.37.tar.gz checksum OK
> + cd /usr/src/lfs
> + make -f pcre LFS_BASEDIR=/usr/src install
> ====================================== Installing pcre-8.37 ...
> Install started; saving file list to /usr/src/lsalr ...
> cd /usr/src/pcre-8.37 && patch -Np1 < 
> /usr/src/src/patches/pcre-8.37-Fix-buffer-overflow-for-named
> -recursive-back-referen.patch
> patching file pcre_compile.c
> patching file testdata/testinput2
> patching file testdata/testoutput2
> cd /usr/src/pcre-8.37 && patch -Np1 < 
> /usr/src/src/patches/pcre-8.37-Fix-buffer-overflow-for-forward
> -reference-within-bac.patch
> patching file pcre_compile.c
> patching file testdata/testinput2
> patching file testdata/testoutput2
> cd /usr/src/pcre-8.37 && patch -Np1 < 
> /usr/src/src/patches/pcre-8.37-Fix-another-buffer-overflow.patch
> patching file pcre_compile.c
> Hunk #1 FAILED at 7210.
> 1 out of 1 hunk FAILED -- saving rejects to file pcre_compile.c.rej
> patching file testdata/testinput2
> patching file testdata/testoutput11-16
> patching file testdata/testoutput11-32
> patching file testdata/testoutput11-8
> patching file testdata/testoutput2
> make: *** [/usr/src/log/pcre-8.37] Error 1
> ***SNAP***
> 
> 'pcre_compile.c.rej' says:
> 
> ***SNIP***
> --- pcre_compile.c
> +++ pcre_compile.c
> @@ -7210,7 +7210,12 @@
>             real compile this will be picked up and the reference 
> wrapped with
>             OP_ONCE to make it atomic, so we must space in case this 
> occurs. */
> 
> -          if (recno == 0) *lengthptr += 2 + 2*LINK_SIZE;
> +          /* In fact, this can happen for a non-forward reference
> because
> +          another group with the same number might be created later.
> This
> +          issue is fixed "properly" in PCRE2. As PCRE1 is now in 
> maintenance
> +          only mode, we finesse the bug by allowing more memory
> always. */
> +
> +          /* if (recno == 0) */ *lengthptr += 2 + 2*LINK_SIZE;
>             }
> 
>           /* In the real compile, search the name table. We check the
> name
> ***SNAP***
> 
> Deleting this section didn't help. Anyone else or my fault?
> 
> Regards
> Matthias

--===============6729037269499585009==
Content-Type: application/pgp-signature
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="signature.asc"
MIME-Version: 1.0

LS0tLS1CRUdJTiBQR1AgU0lHTkFUVVJFLS0tLS0KVmVyc2lvbjogR251UEcgdjEKCmlRSWNCQUFC
Q2dBR0JRSlYxTm85QUFvSkVJQjU4UDl2a0FrSGxtUVAvMDdSOXpoSWJwM09aWXJIb2t0bVFQUzgK
VlJtNExseGVPZnpIeFA3TlArcmxXYTlnWENKWGl1NnpoUUMxcUg0NXBWLzg2TXJZNzVTNmRxb1ZR
emtzY2xiQgpTbEtNaFRuaHpWM0tXZ0I0cWNrZDU2VXBZd1BCem1PaWVZVE1nbGI5MFZtc2R1RWk1
dHBwcklMdFNucUJCWkNRCjJObnNFTjVXdWpqMjRSSEZhTkRmR1BNUDhrN21ZR25Sa2Q2ekZLa3Jv
VEVyMzE2azRCTUNxdGI3OVcyRXc0UzEKV1ZZRlBaNXNwN200SlZOWEQwSmFNR0tBZHhsT0IvNStk
TjVMaUo4K2h4ZFlxdmRyV3M3MVB6RlRJVXJ0Tk9HKwpUVEFySUh6R3pEU3FoTFc3S3FoWTROOWVN
TGorRW1hVFBUUmp1dDY1eW1BRkowZXFiTUwyQ012WXZMNFNzMWlyCjV3MkwyeC9uQ0hpTVJ3MVdT
UVpOVnczQ3ZHV1NDbkE5YVErYllFNnUwbjF3SzhuWGhZMHlCQnAwUHdISVhqdUkKMmRtVlBNRUI3
OUxySDlpZEdkbThXdm1SZkpmR1NoSVkyRTVQZVJ3Q0Nybk1rbVBWc2ROMVR1K1ZET283RkEzZwpC
L01sekxWaTZmVU1VVExLRW9lK2hQU2xBc1BwaXFYWW14WG5yTkZrdDh2T3djeStpSHBINDlHbVlq
QjVJZ2ZCCmx6bFUvT1VPYlRQS3BHcndHUE41UG4vQ1d1dzVoSFdBcUJnOWhGNlZhSTBGdytDQThl
ZHpsRVFid1E5MWh3RTcKNmtXcXdiSForNEJ2MXV5MlZRVGEzZ1RHb2o2VUo4R3dKZjdiZTYyVzdZ
ZU01SUczeFdQcXl3QkY0dWRDVUdpUQpqWHdZTnlrbmtOSkNtYTFPTzIydwo9U1FIKwotLS0tLUVO
RCBQR1AgU0lHTkFUVVJFLS0tLS0K

--===============6729037269499585009==--