Differential D43124

kldxref: Tidy error handling for invalid addresses
ClosedPublic
Actions

Authored by jhb on Dec 19 2023, 8:15 PM.

Tags

None

Referenced Files

	F132037004: D43124.diff
	Mon, Oct 13, 3:37 AM

	Unknown Object (File)
	Fri, Oct 3, 1:38 AM

	Unknown Object (File)
	Wed, Sep 17, 11:38 AM

	Unknown Object (File)
	Wed, Sep 17, 11:03 AM

	Unknown Object (File)
	Tue, Sep 16, 9:50 PM

	Unknown Object (File)
	Aug 28 2025, 11:10 AM

	Unknown Object (File)
	Aug 28 2025, 10:54 AM

	Unknown Object (File)
	Aug 14 2025, 11:28 PM

View All 79 Files

Subscribers

Details

Reviewers

jrtc27
imp

Commits

rGf7e181afba92: kldxref: Tidy error handling for invalid addresses
rG6de6c4211bdd: kldxref: Tidy error handling for invalid addresses
rG6d46e2e0a7f3: kldxref: Tidy error handling for invalid addresses

Summary

Don't print an offset value that's always zero.

ef_get_offset can't return -1.

Sponsored by: DARPA

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Dec 19 2023, 8:15 PM

Herald added a subscriber: imp. · View Herald TranscriptDec 19 2023, 8:15 PM

jhb requested review of this revision.Dec 19 2023, 8:15 PM

Harbormaster completed remote builds in B55034: Diff 131591.Dec 19 2023, 8:15 PM

jhb added a child revision: D43125: kldxref: Simplify elf_read_raw_data.Dec 19 2023, 8:15 PM

imp accepted this revision.Dec 21 2023, 11:12 PM

This revision is now accepted and ready to land.Dec 21 2023, 11:12 PM

Closed by commit rG6d46e2e0a7f3: kldxref: Tidy error handling for invalid addresses (authored by jhb). · Explain WhyDec 22 2023, 3:50 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG6d46e2e0a7f3: kldxref: Tidy error handling for invalid addresses.

jhb added a commit: rG6de6c4211bdd: kldxref: Tidy error handling for invalid addresses.Jan 18 2024, 10:26 PM

jhb added a commit: rGf7e181afba92: kldxref: Tidy error handling for invalid addresses.

Revision Contents
Changeset List

Path

Size

usr.sbin/

kldxref/

6 lines

Diff 131731

usr.sbin/kldxref/ef.c

Loading...