Bug 230903 - FEAT: dwfl_module_addrname() should also return lowpc+highpc
FEAT: dwfl_module_addrname() should also return lowpc+highpc
Status: CLOSED CURRENTRELEASE
Product: Fedora
Classification: Fedora
Component: elfutils (Show other bugs)
rawhide
i686 Linux
medium Severity medium
: ---
: ---
Assigned To: Roland McGrath
:
Depends On:
Blocks: 173278 229424
  Show dependency treegraph
 
Reported: 2007-03-04 10:32 EST by Jan Kratochvil
Modified: 2007-11-30 17:11 EST (History)
3 users (show)

See Also:
Fixed In Version: 0.127-1
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2007-04-19 14:28:20 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Jan Kratochvil 2007-03-04 10:32:27 EST
Description of problem:
If DIE is not available but dwfl_module_addrname() is still able to locate the
symbol it returns only its name.  It could definitely return its base address
(lowpc) and according to ST_SIZE != 0 it could also return its highpc.

Version-Release number of selected component (if applicable):
elfutils-0.126-1.i386

How reproducible:
Always.

Steps to Reproduce:
1. name = dwfl_module_addrname (mod, 0xdeadf00d);

Actual results:
name

Expected results:
GElf_Addr lowpc, highpc;
name = dwfl_module_addrname (mod, 0xdeadf00d, &lowpc, &highpc);
if (name != NULL)
  process_lowpc (lowpc);
if (name != NULL && highpc != 0)
  process_highpc (highpc);

Additional info:
Patching dwfl_module_addrname() is trivia but some elfutils-wise API proposal
would be nice.
Comment 1 Roland McGrath 2007-03-04 16:43:58 EST
I can add a similar call that takes an address but returns full info in the same
way as dwfl_module_getsym.
Comment 2 Jan Kratochvil 2007-03-04 16:59:42 EST
Confirming I would like such API for these needs.  I hope it would
reuse/refactor some of the existing code of
dwfl_module_getsym()/gelf_getsymshndx() there.
Comment 3 Roland McGrath 2007-03-04 17:43:46 EST
Of course it was already using that internally.  I've added the new
dwfl_module_addrsym call upstream, it was trivial.
Comment 4 Nurdin Premji 2007-03-26 13:29:43 EDT
Is the patch available?
Comment 5 Roland McGrath 2007-04-19 14:28:20 EDT
0.127 includes dwfl_module_addrsym

Note You need to log in before you can comment on or make changes to this bug.