From: Alexander Cherepanov <ch3root@openwall.com>
To: musl@lists.openwall.com
Subject: Re: [musl] Minor style patch to exit.c
Date: Sun, 19 Jan 2020 18:53:49 +0300 [thread overview]
Message-ID: <ad60275b-fd62-2e38-7a17-a64f4bc0a9f0@openwall.com> (raw)
In-Reply-To: <20200119142401.GG2020@voyager>
On 19/01/2020 17.24, Markus Wichmann wrote:
> On Sun, Jan 19, 2020 at 04:33:47PM +0300, Alexander Cherepanov wrote:
>> Couldn't _start defined as an array? Then separate values could be accessed
>> simply as elements of this array. And casts to integers could be limited to
>> calculating the number of elements, the terminating value or something.
>
> That reminds me of something I read in the C standard: Two pointers must
> compare equal if, among other possibilities, one is a pointer to
> one-past its underlying array, and the other is a pointer to the start
> of its array, and the arrays happen to lie behind one another in address
> space.
One[1] of the gcc bug reports I mentioned is exactly about this issue.
DR 260[2] allows to take the provenance of the pointers into account
when comparing them and gcc really does this.
[1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61502
[2] http://open-std.org/jtc1/sc22/wg14/www/docs/dr_260.htm
As a side note, I thinks this is the wildest gcc bug report, it contains
really mind-blowing comments (like comment 3). I don't mean it in a bad
way at all and if you want to turn your understanding of C language
inside-out you can try to read it. OTOH I think it's all wrong after all
and I have some hope for it to be settled after my recent comments
there. But I don't hold my breath.
> Therefore, if _start and _end were arrays, even the GCC devs must agree
> that there might be an integer i such that _start + i == _end. For the C
> language, _start and _end would be arrays that happen to lie adjacent in
> address space.
>
> And if we have guarantees from the outside attesting to that, then
> _end - _start is no longer an undefined expression, right?
Even if we know that _start + k == _end it doesn't mean that we allowed
to subtract them.
--
Alexander Cherepanov
next prev parent reply other threads:[~2020-01-19 15:54 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-19 11:07 Markus Wichmann
2020-01-19 11:12 ` Markus Wichmann
2020-01-19 11:31 ` Szabolcs Nagy
2020-01-19 12:17 ` Markus Wichmann
2020-01-19 13:33 ` Alexander Cherepanov
2020-01-19 14:24 ` Markus Wichmann
2020-01-19 14:49 ` Pascal Cuoq
2020-01-19 15:53 ` Alexander Cherepanov [this message]
2020-01-19 16:22 ` Rich Felker
2020-01-19 21:02 ` Alexander Cherepanov
2020-01-19 14:46 ` Alexander Monakov
2020-01-19 16:18 ` Rich Felker
2020-01-19 17:11 ` Alexander Monakov
2020-01-19 17:17 ` Alexander Monakov
2020-01-19 17:19 ` Rich Felker
2020-01-19 17:32 ` Alexander Monakov
2020-01-19 17:38 ` Rich Felker
2020-01-19 19:13 ` Alexander Monakov
2020-01-19 16:33 ` Alexander Cherepanov
2020-01-19 16:39 ` Rich Felker
2020-01-19 21:34 ` Alexander Cherepanov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ad60275b-fd62-2e38-7a17-a64f4bc0a9f0@openwall.com \
--to=ch3root@openwall.com \
--cc=musl@lists.openwall.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/musl/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).