Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Created attachment 15453
--> https://sourceware.org/bugzilla/attachment.cgi?id=15453=edit
proposed fix
The sh linker's handling of function descript
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
This looks like it may be the same as
https://sourceware.org/bugzilla/show_bug.cgi?id=22374 for other archs. I was
testing with the program:
#include
static void
Severity: normal
Priority: P2
Component: gold
Assignee: ccoutant at gmail dot com
Reporter: bugdal at aerifal dot cx
CC: ian at airs dot com
Target Milestone: ---
When targeting powerpc64, attempting to link a source file that calls
https://sourceware.org/bugzilla/show_bug.cgi?id=26541
--- Comment #5 from Rich Felker ---
Ah no, at least according to your patch the original gold bug seems to be there
still too.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=26541
--- Comment #2 from Rich Felker ---
Why is this still open? It's trivial to fix.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #13 from Rich Felker ---
This patch keeps expanding way beyond the scope of what I can commit to review,
and I don't think it's nearing something that fixes the problem. It's not clear
to me if bfd_stat uses fstat on the open file
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #11 from Rich Felker ---
Yes, see my footnote in comment 4. That works just as well.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #9 from Rich Felker ---
> > 1. make_tempname with fd_return==NULL is *always* a bug,
> Well, I would argue that it is only a bug if the temporary file is going to
be manipulated by other system functions.
Are you going to create
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #4 from Rich Felker ---
That's a lot to review without being familiar with the code, but a couple
things I can tell you right off:
1. make_tempname with fd_return==NULL is *always* a bug, and defeats the whole
purpose of mkstemp.
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
--- Comment #2 from Rich Felker ---
It's not that smart_rename should open it and use fchown+fchmod on the result;
indeed as you figured that has the same race. The problem is that you threw
away the only safe way to perform modifications on
https://sourceware.org/bugzilla/show_bug.cgi?id=26945
Rich Felker changed:
What|Removed |Added
CC||siddhesh at sourceware dot org
--
You
Component: binutils
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
At least objcopy and perhaps other utilities generate a temp file safely with
mkstemp then rename it to atomically replace the original
https://sourceware.org/bugzilla/show_bug.cgi?id=25882
--- Comment #9 from Rich Felker ---
OK, I should have read more before commenting. My comments in particular
pertain specifically to powerpc64 although other archs might be affected too. I
see that the error was downgraded to a warning since
https://sourceware.org/bugzilla/show_bug.cgi?id=25882
--- Comment #8 from Rich Felker ---
Also note that while, for musl targets, this issue could also be fixed just by
getting gcc not to build its ld128 functions in libgcc_s, there are also people
using glibc's ld64 ABI, and glibc necessarily
https://sourceware.org/bugzilla/show_bug.cgi?id=25882
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Priority: P2
Component: gold
Assignee: ccoutant at gmail dot com
Reporter: bugdal at aerifal dot cx
CC: ian at airs dot com
Target Milestone: ---
In gold's output.cc gold_fallocate, after posix_fallocate() fails with a
non-fatal error
https://sourceware.org/bugzilla/show_bug.cgi?id=23825
--- Comment #8 from Rich Felker ---
To clarify what I mean about ABI and copy relocations, prior to RISC-V doing
this, there was no arch where it was unsafe to have extensible (existing part's
meaning remains same, new parts added to end)
https://sourceware.org/bugzilla/show_bug.cgi?id=23825
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Component: gas
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
I don't understand the lisp-like language the opcode C files are generated
from, so I don't have a root cause for this, but the problem is in
opcodes/or1k
https://sourceware.org/bugzilla/show_bug.cgi?id=25184
Rich Felker changed:
What|Removed |Added
Target||or1k-*-*
--
You are receiving this
https://sourceware.org/bugzilla/show_bug.cgi?id=25056
--- Comment #3 from Rich Felker ---
It is a correctness issue for static linking, where my intent (as the one who
introduced static pie) was that presence of any unresolved relocations except
load-address-relative ones is an error. If you
https://sourceware.org/bugzilla/show_bug.cgi?id=25056
--- Comment #1 from Rich Felker ---
Link for convenience: https://sourceware.org/bugzilla/show_bug.cgi?id=22570
I suspect there may be other targets with exactly the same issue. There should
probably be a generic test for all targets that
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Created attachment 12015
--> https://sourceware.org/bugzilla/attachment.cgi?id=12015=edit
patch to fix
Identical to #22570 for mips.
--
https://sourceware.org/bugzilla/show_bug.cgi?id=23683
--- Comment #2 from Rich Felker ---
Thanks. I was still using 2.27 and should have tagged that in the report, and
should have checked later history for a fix.
--
You are receiving this mail because:
You are on the CC list for the bug.
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
In the help text (ld --help), -z stack-size=SIZE is misspelled as -z
stacksize=SIZE, which does not work.
--
You are receiving this mail because:
You
https://sourceware.org/bugzilla/show_bug.cgi?id=23428
--- Comment #3 from Rich Felker ---
I don't object to the patch as-is, but wouldn't it make more sense to make the
program headers a first-class section with attribute alloc, so that they're
semantically forced to be visible at runtime rather
https://sourceware.org/bugzilla/show_bug.cgi?id=23428
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--
You
https://sourceware.org/bugzilla/show_bug.cgi?id=22263
--- Comment #17 from Rich Felker ---
Yes, see #22570 in the list of duplicates, which includes a proposed patch for
MIPS.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=22570
--- Comment #2 from Rich Felker ---
Created attachment 10762
--> https://sourceware.org/bugzilla/attachment.cgi?id=10762=edit
proposed fix
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=22570
--- Comment #1 from Rich Felker ---
The details of my assessment seem to have been written in terms of 2.25; with
the changes from commit 0e1862bb40, the problem is now use of bfd_link_pic
where bfd_link_dll was presumably intended. I'm
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
When a symbol is defined and referenced in the main program, DTPMOD relocations
should be bound at ld time to module 1. This is mandatory for static
https://sourceware.org/bugzilla/show_bug.cgi?id=22570
Rich Felker changed:
What|Removed |Added
Target||mips*-*-*-*
--
You are receiving this
https://sourceware.org/bugzilla/show_bug.cgi?id=21017
--- Comment #1 from Rich Felker ---
Created attachment 9735
--> https://sourceware.org/bugzilla/attachment.cgi?id=9735=edit
patch to fix the bug
This seems to fix the issue, but there may be additional reloc types that
should be checked to
Status: UNCONFIRMED
Severity: normal
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
When linking a program where all GOT-based relocations are GOT-relative
https://sourceware.org/bugzilla/show_bug.cgi?id=17699
--- Comment #7 from Rich Felker ---
Can you cite any documentation that a non-STT_TLS symbol reference is
permissible in a DTPMOD relocation?
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=17739
--- Comment #11 from Rich Felker ---
Ping?
--
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=20360
--- Comment #6 from Rich Felker ---
To add some context, I suggested building without -Bsymbolic* in case the cause
was relocations that would be textrels before -Bsymbolic* optimized them out.
Looking at the source, particularly calls/jmps
https://sourceware.org/bugzilla/show_bug.cgi?id=20360
--- Comment #3 from Rich Felker ---
At this point there's no testcase smaller than the dotnet/coreclr build, but I
wanted to go ahead and post the bug here to get discussion moved to the tracker
where it's relevant. I believe they have a way
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
This is based on a bug report from dotnet CoreCLR:
https://github.com/dotnet/coreclr/issues/6211
What seems to be happening
https://sourceware.org/bugzilla/show_bug.cgi?id=17739
Rich Felker changed:
What|Removed |Added
CC||bugdal at aerifal dot cx
--- Comment
Severity: normal
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
If binutils is built with STT_COMMON support enabled, use of
-Bsymbolic-functions for libraries with visible data
https://sourceware.org/bugzilla/show_bug.cgi?id=19516
Rich Felker changed:
What|Removed |Added
Target||microblaze*-*-*
--
You are receiving
https://sourceware.org/bugzilla/show_bug.cgi?id=19516
--- Comment #2 from Rich Felker ---
Created attachment 8978
--> https://sourceware.org/bugzilla/attachment.cgi?id=8978=edit
proposed fix
Producing a symbolic GLOB_DAT relocation for a symbol that has been made
local/hidden by setting
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
Rich Felker changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Current git versions of musl libc (since commit ad1cd43a86) are producing a
libc.so that crashes during startup due to a bogus GLOB_DAT relocations with
symbol
https://sourceware.org/bugzilla/show_bug.cgi?id=19516
--- Comment #1 from Rich Felker ---
For reference here is the link to the first version of musl affected which can
be used to reproduce the bug:
http://git.musl-libc.org/cgit/musl/commit/?id=ad1cd43a86645ba2d4f7c8747240452a349d6bc1
--
You
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
--- Comment #4 from Rich Felker ---
OK. Should I submit a patch or do you just want to make the one-line change
yourself?
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
--- Comment #2 from Rich Felker ---
Even with no GOT, not PLT, and no function descriptors, it's possible to have
code which accesses local data via @GOTOFF displacements. If DT_PLTGOT is not
emitted in the output _DYNAMIC, there is no way
https://sourceware.org/bugzilla/show_bug.cgi?id=19091
Rich Felker changed:
What|Removed |Added
CC||kkojima at rr dot iij4u.or.jp
--
You
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Created attachment 8688
--> https://sourceware.org/bugzilla/attachment.cgi?id=8688=edit
minimal test case
Under some circumstan
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #13 from Rich Felker bugdal at aerifal dot cx ---
Nick, can you confirm that your latest patch does not break typical use of
.size like Alan Modra was concerned it would?
--
You are receiving this mail because:
You are on the CC
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #15 from Rich Felker bugdal at aerifal dot cx ---
Well that's not necessarily a show-stopper if there's a way to make it work
(evaluate at as-time) in .size or other contexts that aren't actually section
content and where
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #11 from Rich Felker bugdal at aerifal dot cx ---
I filed the corresponding GCC bug, #66609, at the same time I filed this one.
Here is the link for reference:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66609
The bug report
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #9 from Rich Felker bugdal at aerifal dot cx ---
If this is the intended behavior, it should be documented as such. At the very
least it's highly counter-intuitive, and a regression, albeit a very very old
one. In any case I'll
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #4 from Rich Felker bugdal at aerifal dot cx ---
Indeed, that fixes it. However I'm skeptical of the way you're making the
change:
1. S_FORCE_RELOC already has logic to force relocations for weak symbols when
the strict argument
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #7 from Rich Felker bugdal at aerifal dot cx ---
I like it, but I think some effort should be put into understanding why the
wrong code was there to begin with and what the intent was. Here is the commit
that broke it:
https
https://sourceware.org/bugzilla/show_bug.cgi?id=18561
--- Comment #2 from Rich Felker bugdal at aerifal dot cx ---
No. I believe I actually tried something near-identical before reporting this
bug, but I just tried your patch to be sure, and there is still no relocation
in the output object file
Severity: normal
Priority: P2
Component: gas
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Target Milestone: ---
Minimal test case:
.global a
.global b
.weak a
.align 2
a: .long 0
b: .long a-.
The output should contain
https://sourceware.org/bugzilla/show_bug.cgi?id=16858
Rich Felker bugdal at aerifal dot cx changed:
What|Removed |Added
Status|RESOLVED|REOPENED
https://sourceware.org/bugzilla/show_bug.cgi?id=17699
Rich Felker bugdal at aerifal dot cx changed:
What|Removed |Added
CC||bugdal
https://sourceware.org/bugzilla/show_bug.cgi?id=16698
Rich Felker bugdal at aerifal dot cx changed:
What|Removed |Added
CC||bugdal
Priority: P2
Component: ld
Assignee: unassigned at sourceware dot org
Reporter: bugdal at aerifal dot cx
Attempting to compile/link the following program:
__thread int my_tls = 1;
int *my_tls_addr() { return my_tls; }
with the following gcc command line:
gcc
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
--- Comment #9 from Rich Felker bugdal at aerifal dot cx 2013-01-14 15:50:47
UTC ---
I agree there's room for different opinions here, but I believe --as-needed
should not include the library when all it does is resolve weak references
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
--- Comment #11 from Rich Felker bugdal at aerifal dot cx 2013-01-14 17:22:10
UTC ---
I agree that's possible as a workaround, but it seems to change the ABI for
libgcc_s, which is probably not acceptable. And it's an extra layer of hackery
http://sourceware.org/bugzilla/show_bug.cgi?id=14798
Bug #: 14798
Summary: gas mips target generates badly typed symbols
Product: binutils
Version: unspecified
Status: NEW
Severity: normal
Priority: P2
http://sourceware.org/bugzilla/show_bug.cgi?id=13600
--- Comment #15 from Rich Felker bugdal at aerifal dot cx 2012-10-12 19:48:59
UTC ---
Ping. Is anybody willing to look at this?
This bug has been open 10 months now and fixing it is as simple as removing the
offending code that's
http://sourceware.org/bugzilla/show_bug.cgi?id=14640
Bug #: 14640
Summary: gas generates broken relocations for symbols with weak
defs on microblaze
Product: binutils
Version: unspecified
Status: NEW
http://sourceware.org/bugzilla/show_bug.cgi?id=14512
Bug #: 14512
Summary: -z nodelete should be default for shared libraries
Product: binutils
Version: unspecified
Status: NEW
Severity: normal
Priority: P2
http://sourceware.org/bugzilla/show_bug.cgi?id=13600
Rich Felker bugdal at aerifal dot cx changed:
What|Removed |Added
CC||bugdal
http://sourceware.org/bugzilla/show_bug.cgi?id=14156
Bug #: 14156
Summary: --sort-section=alignment trashes init/fini sections,
and anything similar
Product: binutils
Version: unspecified
Status: NEW
Severity:
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
Rich Felker bugdal at aerifal dot cx changed:
What|Removed |Added
Component|ld |binutils
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
--- Comment #3 from Rich Felker bugdal at aerifal dot cx 2012-05-01 18:37:16
UTC ---
Another way of looking at the issue is that, if strip is going to break things
like this, the default linker scripts should order the sections
http://sourceware.org/bugzilla/show_bug.cgi?id=14037
Bug #: 14037
Summary: Bogus global symbol ABS redefinition when linking PIC
app to eh_frame-stripped shared library
Product: binutils
Version: unspecified
Status: NEW
http://sourceware.org/bugzilla/show_bug.cgi?id=13891
--- Comment #4 from Rich Felker bugdal at aerifal dot cx 2012-03-24 21:06:21
UTC ---
Mike, a link to the workaround would be really helpful to anyone dealing with
this bug or trying to evaluate whether it's worth fixing/working around
http://sourceware.org/bugzilla/show_bug.cgi?id=12549
Rich Felker bugdal at aerifal dot cx changed:
What|Removed |Added
CC||bugdal
http://sourceware.org/bugzilla/show_bug.cgi?id=13891
Bug #: 13891
Summary: Binutils is impossible to static link due to broken
libtool behavior
Product: binutils
Version: unspecified
Status: NEW
Severity:
76 matches
Mail list logo