This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [Patch] ld: check for address space overflow

From: Tristan Gingold <gingold at adacore dot com>
To: Alan Modra <amodra at gmail dot com>
Cc: "H.J. Lu" <hjl dot tools at gmail dot com>, binutils <binutils at sourceware dot org>
Date: Wed, 15 Mar 2017 10:26:02 +0100
Subject: Re: [Patch] ld: check for address space overflow
Authentication-results: sourceware.org; auth=none
References: <0C5EB7A7-DFB1-404C-A976-E96FC881B89A@adacore.com> <CAMe9rOov9FxvPs6NHW1Kgnu8ug+tOc4jWZrjmKy-o61WP0Y3QQ@mail.gmail.com> <CAMe9rOqwDo3nzhLY4jDYabfYGAj9dFx3kUBkf2D0en3tu7r6HA@mail.gmail.com> <F639AA22-1566-43A8-96DD-C3013A5E5B5B@adacore.com> <20170314101952.GL4983@bubble.grove.modra.org> <32FDC254-53F6-472C-887A-8FAED5D24CB7@adacore.com> <20170314121729.GM4983@bubble.grove.modra.org> <205F4E12-577B-47D7-B382-E33F1F1173AB@adacore.com> <20170315011101.GN4983@bubble.grove.modra.org>

> On 15 Mar 2017, at 02:11, Alan Modra <amodra@gmail.com> wrote:
> 
> On Tue, Mar 14, 2017 at 02:56:18PM +0100, Tristan Gingold wrote:
>>    	* testsuite/ld-checks/checks.exp (overflow_check): Disable for
>>    	non-elf targets.
> 
> OK.

Thanks, committed.

>  I'm going to commit the following too.  h8300 and ip2k targets
> use a 16-bit address cpu by default, so hit the address wrap check
> (more than once) on the 64k section test.
> 
> 	* testsuite/ld-elf/sec64k.exp: Don't run on h8300 and ip2k.

Thank you for the head-up.
Maybe the overflow check should also check that the length is not larger than the address space (otherwise it could have missed these h8300 and ip2k cases), but I think this is a too artificial case.

Tristan.

References:
- [Patch] ld: check for address space overflow
  - From: Tristan Gingold
- Re: [Patch] ld: check for address space overflow
  - From: H.J. Lu
- Re: [Patch] ld: check for address space overflow
  - From: H.J. Lu
- Re: [Patch] ld: check for address space overflow
  - From: Tristan Gingold
- Re: [Patch] ld: check for address space overflow
  - From: Alan Modra
- Re: [Patch] ld: check for address space overflow
  - From: Tristan Gingold
- Re: [Patch] ld: check for address space overflow
  - From: Alan Modra
- Re: [Patch] ld: check for address space overflow
  - From: Tristan Gingold
- Re: [Patch] ld: check for address space overflow
  - From: Alan Modra

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]