Page MenuHomeFreeBSD
Differential D55259

ngctl: Fix buffer overflow in config command
ClosedPublic
Actions

Authored by des on Feb 12 2026, 12:36 AM.
Tags
None
Referenced Files
F163291143: D55259.id171869.diff
Tue, Jul 21, 7:05 PM
Unknown Object (File)
Mon, Jul 20, 3:49 PM
Unknown Object (File)
Sun, Jul 19, 8:17 PM
Unknown Object (File)
Sat, Jul 18, 7:57 PM
Unknown Object (File)
Wed, Jul 15, 7:07 AM
Unknown Object (File)
Wed, Jul 15, 7:07 AM
Unknown Object (File)
Tue, Jul 14, 6:56 AM
Unknown Object (File)
Sat, Jul 11, 5:31 PM
View All 56 Files
Subscribers
donner
imp
zlei

Details

Reviewers
kevans
markj
zlei
Commits
rGf16fc39527ee: ngctl: Fix buffer overflow in config command
rG2b11bbd790ac: ngctl: Fix buffer overflow in config command
rGe5bf728058da: ngctl: Fix buffer overflow in config command
rG669347f67a07: ngctl: Fix buffer overflow in config command
rG59906a163e47: ngctl: Fix buffer overflow in config command
Summary

Keep track of our buffer length when assembling the argument list.

PR: 293075
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70634
Build 67517: arc lint + arc unit

Event Timeline

des created this revision.Feb 12 2026, 12:36 AM
Herald added subscribers: donner, imp. · View Herald TranscriptFeb 12 2026, 12:36 AM
des requested review of this revision.Feb 12 2026, 12:36 AM
Harbormaster completed remote builds in B70634: Diff 171805.Feb 12 2026, 12:36 AM
des added a parent revision: D55258: ngctl: Check hook name length.Feb 12 2026, 12:36 AM
markj accepted this revision.Feb 12 2026, 2:45 PM
This revision is now accepted and ready to land.Feb 12 2026, 2:45 PM
markj mentioned this in D55254: ngctl: Check for overflow when assembling a config command.Feb 12 2026, 2:46 PM
zlei accepted this revision.Feb 12 2026, 3:24 PM
zlei added a subscriber: zlei.

Solid fix !

Closed by commit rG59906a163e47: ngctl: Fix buffer overflow in config command (authored by des). · Explain WhyFeb 13 2026, 4:02 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG59906a163e47: ngctl: Fix buffer overflow in config command.
des added a commit: rG669347f67a07: ngctl: Fix buffer overflow in config command.Feb 17 2026, 11:58 PM
des added a commit: rGe5bf728058da: ngctl: Fix buffer overflow in config command.Feb 18 2026, 12:09 AM
des added a commit: rG2b11bbd790ac: ngctl: Fix buffer overflow in config command.Feb 18 2026, 12:15 AM
cperciva added a commit: rGf16fc39527ee: ngctl: Fix buffer overflow in config command.Feb 18 2026, 1:54 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
ngctl/
22 lines

Diff 171805

View Options

usr.sbin/ngctl/config.c

Loading...