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

	F107155844: D43124.id131591.diff
	Fri, Jan 10, 11:40 PM

	F107150586: D43124.diff
	Fri, Jan 10, 9:40 PM

	Unknown Object (File)
	Thu, Dec 26, 3:55 PM

	Unknown Object (File)
	Dec 8 2024, 3:58 AM

	Unknown Object (File)
	Dec 8 2024, 3:58 AM

	Unknown Object (File)
	Dec 8 2024, 3:58 AM

	Unknown Object (File)
	Dec 8 2024, 3:44 AM

	Unknown Object (File)
	Nov 9 2024, 7:51 PM

View All 54 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...