[PATCH] Let libelf.h's loff_t fall back to off_t

Josh Stone jistone at redhat.com
Fri Oct 9 21:24:18 UTC 2015


On 10/09/2015 01:57 PM, Roland McGrath wrote:
> Hmm.  Is it the case that loff_t or off_t that we want to use will always
> be 64 bits?  The way we use it in the libelf interfaces always wants a
> positive offset (or size).  I don't think change of signedness really needs
> to count as an ABI change here.  Could we just use Elf64_Off and be done
> with it?

It was explicitly off64_t until your commit 1a2e8f4f5bd2.  And yes, as
written I expect either loff_t or off_t to always be 64 bits too.
(I think technically loff_t is "at least" 64 bits.)

However, I'm afraid to change signedness, if only because -1 is used in
error conditions, e.g. returned from elf_update().  Someone checking <0
will be in for a bad time.

How about just int64_t?  We already pull stdint.h for elf.h, so it seems
fine to include and use that in libelf.h too.


More information about the elfutils-devel mailing list