diff options
| author | blasty <blasty@local> | 2026-07-09 22:28:53 +0200 |
|---|---|---|
| committer | blasty <blasty@local> | 2026-07-09 22:28:53 +0200 |
| commit | 6a761c222a6f08cb2fd825a15f3d3f44390a716d (patch) | |
| tree | 8a0a6c3add4eab61555783dd407a7c2aa83070ec /idatui/domain.py | |
| parent | decompiler: xref jump lands the cursor on the reference token's column (diff) | |
| download | ida-tui-6a761c222a6f08cb2fd825a15f3d3f44390a716d.tar.gz ida-tui-6a761c222a6f08cb2fd825a15f3d3f44390a716d.tar.xz ida-tui-6a761c222a6f08cb2fd825a15f3d3f44390a716d.zip | |
xrefs: informative labels — function+offset, and section instead of '?'
The xref list showed just the containing function name, so multiple sites in the
same function were indistinguishable ('sub_2300' repeated), and references not in
any function collapsed to a bare '?'.
- In-function sites now show fn_name+offset (e.g. main+0xb08), so each site is
distinct and you can see where in the function it is.
- Non-function sites (GOT/reloc data slots, loose thunks) show the section they
live in (.got, .data.rel.ro, .text, LOAD, ...) instead of '?'. Add
Program.sections()/section_of() over survey_binary's segment map (fetched once,
cached lazily).
Pilot checks for both. full suite 67/67.
Diffstat (limited to 'idatui/domain.py')
| -rw-r--r-- | idatui/domain.py | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/idatui/domain.py b/idatui/domain.py index 4902a64..cdbdd67 100644 --- a/idatui/domain.py +++ b/idatui/domain.py @@ -376,6 +376,7 @@ class Program: self._disasm: dict[int, DisasmModel] = {} self._decomp: dict[int, tuple[Decompilation, int]] = {} self._name_gen = 0 # bumped on rename; invalidates stale name caches + self._sections: list[tuple[int, int, str]] | None = None self._lock = threading.Lock() # -- prefetch plumbing ------------------------------------------------- # @@ -397,6 +398,39 @@ class Program: self._indices[filter] = idx return idx + # -- sections / segments ---------------------------------------------- # + def sections(self) -> list[tuple[int, int, str]]: + """Sorted, non-overlapping [(start, end, name)] segment map (cached). + Fetched once from ``survey_binary`` (~0.7s) on first use.""" + if self._sections is not None: + return self._sections + secs: list[tuple[int, int, str]] = [] + try: + sb = self.client.call("survey_binary") + for s in (sb.get("segments", []) if isinstance(sb, dict) else []): + try: + secs.append((_as_int(s["start"]), _as_int(s["end"]), + s.get("name", ""))) + except (KeyError, ValueError, TypeError): + continue + except Exception: # noqa: BLE001 -- best-effort; callers handle None + secs = [] + secs.sort() + with self._lock: + self._sections = secs + return secs + + def section_of(self, ea: int) -> str | None: + """Name of the segment/section containing ``ea`` (e.g. '.got', '.text', + '.data.rel.ro', 'LOAD'), or None if unmapped.""" + secs = self.sections() + if not secs: + return None + i = bisect.bisect_right([s[0] for s in secs], ea) - 1 + if 0 <= i < len(secs) and secs[i][0] <= ea < secs[i][1]: + return secs[i][2] + return None + # -- disassembly ------------------------------------------------------- # def disasm(self, ea: int, name: str | None = None) -> DisasmModel: with self._lock: |
