Page MenuHomeFreeBSD
Differential D31939

efitable: Don't pass NULL as a format string to xo_err().
ClosedPublic
Actions

Authored by jhb on Sep 13 2021, 6:21 PM.
Tags
None
Referenced Files
F159209729: D31939.id95065.diff
Thu, Jun 11, 9:20 AM
Unknown Object (File)
Wed, Jun 10, 4:28 AM
Unknown Object (File)
Sat, May 16, 2:21 AM
Unknown Object (File)
Sat, May 16, 2:21 AM
Unknown Object (File)
Sat, May 16, 2:21 AM
Unknown Object (File)
May 12 2026, 4:00 PM
Unknown Object (File)
May 5 2026, 11:24 PM
Unknown Object (File)
Apr 30 2026, 6:08 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
emaste
Commits
rGb14cd3a8339e: efitable: Don't pass NULL as a format string to xo_err().
Summary

This fixes a -Wformat error reported by GCC 9.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41493
Build 38382: arc lint + arc unit

Event Timeline

jhb created this revision.Sep 13 2021, 6:21 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 13 2021, 6:21 PM
jhb requested review of this revision.Sep 13 2021, 6:21 PM
Harbormaster completed remote builds in B41493: Diff 95065.Sep 13 2021, 6:21 PM
kib accepted this revision.Sep 13 2021, 6:54 PM
kib added inline comments.
usr.sbin/efitable/efitable.c
149

I suggest to add something like '(len == 0)' into the string, to distinguish two calls.

This revision is now accepted and ready to land.Sep 13 2021, 6:54 PM
jhb marked an inline comment as done.Sep 15 2021, 3:40 PM
Closed by commit rGb14cd3a8339e: efitable: Don't pass NULL as a format string to xo_err(). (authored by jhb). · Explain WhySep 15 2021, 4:06 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGb14cd3a8339e: efitable: Don't pass NULL as a format string to xo_err()..

Revision Contents
Changeset List

PathSize
usr.sbin/
efitable/
4 lines

Diff 95065

View Options

usr.sbin/efitable/efitable.c

Loading...