Page MenuHomeFreeBSD
Differential D46764

libkldelf: add elf_lookup_symbol function
ClosedPublic
Actions

Authored by khng on Sep 23 2024, 6:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 9, 1:13 AM
Unknown Object (File)
Mon, Sep 7, 1:48 PM
Unknown Object (File)
Sun, Sep 6, 7:55 PM
Unknown Object (File)
Sun, Sep 6, 5:23 PM
Unknown Object (File)
Sun, Sep 6, 2:22 AM
Unknown Object (File)
Sat, Sep 5, 1:32 PM
Unknown Object (File)
Sat, Sep 5, 10:57 AM
Unknown Object (File)
Fri, Sep 4, 5:10 AM
View All Files
Subscribers
None

Details

Reviewers
markj
imp
jhb
Commits
rG43628a31e3a2: libkldelf: add elf_lookup_symbol function
rG2c7d84795628: libkldelf: add elf_lookup_symbol function
Summary

The elf_lookup_symbol function looks up the symbol with a given symbol
name. A pointer to the GElf_Sym of the symbol is returned if the symbol
exists in the opened ELF file.

Sponsored by: Juniper Networks, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

khng requested review of this revision.Sep 23 2024, 6:42 PM
khng created this revision.
Harbormaster completed remote builds in B59573: Diff 143646.Sep 23 2024, 6:42 PM
khng added a parent revision: D46719: libkldelf: add a private library for kernel/kld-related ELF parsing.Sep 23 2024, 6:42 PM
markj added inline comments.Sep 27 2024, 10:44 AM
lib/libkldelf/kldelf.h
321

What if there is more than one symbol with the same name? This can happen e.g., with local function symbols.

khng added inline comments.Sep 27 2024, 3:29 PM
lib/libkldelf/kldelf.h
321

For now it will just return the first one being found when traversing symtab.

markj accepted this revision.Sep 30 2024, 8:49 AM
markj added inline comments.
lib/libkldelf/kldelf.h
321

I'd mention that caveat in the comment.

This revision is now accepted and ready to land.Sep 30 2024, 8:49 AM
Closed by commit rG2c7d84795628: libkldelf: add elf_lookup_symbol function (authored by khng). · Explain WhyOct 8 2024, 4:29 AM
This revision was automatically updated to reflect the committed changes.
khng added a commit: rG2c7d84795628: libkldelf: add elf_lookup_symbol function.
khng added a reverting change: rG7007de7d1095: Revert "libkldelf: add elf_lookup_symbol function".Oct 8 2024, 7:49 PM
khng added a commit: rG43628a31e3a2: libkldelf: add elf_lookup_symbol function.Oct 18 2024, 8:22 PM

Revision Contents
Changeset List

PathSize
lib/
libkldelf/
1 line
1 line
6 lines
13 lines

Diff 144446

View Options

lib/libkldelf/ef.c

Loading...
View Options

lib/libkldelf/ef_obj.c

Loading...
View Options

lib/libkldelf/elf.c

Loading...
View Options

lib/libkldelf/kldelf.h

Loading...