Re: [RFC PATCH 0/3] Pretty-printing for errno
Pedro Alves <[email protected]>
| Newsgroups | gmane.comp.lib.glibc.alpha,gmane.comp.gdb.devel |
|---|---|
| Message-ID | <[email protected]> |
On 07/13/2017 03:30 AM, Pedro Alves wrote:
> On 06/30/2017 07:11 PM, Pedro Alves wrote:
>> On 06/30/2017 06:27 PM, Zack Weinberg wrote:
>>> Pedro Alves wrote:
>>>> The next problem is that without debug info for __errno_location,
>>>> gdb has no clue of its prototype, only that its a function, and so
>>>> it assumes that it has type "int()", i.e., that it returns int,
>>>> while in reality it returns int * / __error_t *. (Falling back
>>>> to assuming "int" is IMO not the best idea, but I don't know
>>>> the history behind it.)
>>>
>>> Probably because that's the pre-C89 legacy default function signature?
>>
>> Yes, most likely.
>
> FYI, shortly after this discussion, yet another user showed up
> on #gdb confused by "p getenv("PATH") returning weird negative
> integer numbers [because he had no debug info for getenv...], so I
> decided to do something about it. I've now sent a patch series
> that stops GDB from assuming no-debug-info symbols have
> type int:
>
> [PATCH 00/13] No-debug-info debugging improvements
> https://sourceware.org/ml/gdb-patches/2017-07/msg00112.html
>
> Comments welcome.
FYI, this is now all in gdb master. I believe all the gdb issues
uncovered by the errno printer have been addressed. Let me know
if you're aware of something still not working properly.
Thanks,
Pedro Alves