Paths

Table of Contentst

lpd: Add timeout option
ClosedPublic
Actions

Authored by des on Feb 20 2026, 4:03 PM.

Details

Reviewers

Commits

rGba5f7719f63e: lpd: Add timeout option
rGf413ebdf126d: lpd: Add timeout option
rG041065e59eae: lpd: Add timeout option
rG56fbfd1ecdc7: lpd: Add timeout option

Summary

Set a 120-second receive timeout on all client connections, and add a
command-line option to change that value.

MFC after: 3 days

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

des created this revision.Feb 20 2026, 4:03 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptFeb 20 2026, 4:03 PM

des requested review of this revision.Feb 20 2026, 4:03 PM

Harbormaster completed remote builds in B70820: Diff 172317.Feb 20 2026, 4:03 PM

des added a parent revision: D55399: lpd: Improve robustness.Feb 20 2026, 4:03 PM

des added a reviewer: markj.Feb 20 2026, 4:05 PM

markj accepted this revision.Feb 21 2026, 2:55 AM

markj added inline comments.

usr.sbin/lpr/lpd/lpd.c
393	Should we bother with a timeout in the `domain == AF_UNIX` case?
394

This revision is now accepted and ready to land.Feb 21 2026, 2:55 AM

des added inline comments.Feb 21 2026, 8:50 AM

usr.sbin/lpr/lpd/lpd.c
393	Why should we trust local unprivileged users more than we trust root on an authorized client?

Closed by commit rG56fbfd1ecdc7: lpd: Add timeout option (authored by des). · Explain WhyFeb 26 2026, 6:16 AM

This revision was automatically updated to reflect the committed changes.

des added a commit: rG56fbfd1ecdc7: lpd: Add timeout option.

des added a commit: rG041065e59eae: lpd: Add timeout option.Mar 4 2026, 2:45 PM

des added a commit: rGf413ebdf126d: lpd: Add timeout option.

des added a commit: rGba5f7719f63e: lpd: Add timeout option.

Revision Contents
Changeset List

Path

Size

usr.sbin/

lpr/

lpd/

lpd.8

8 lines

lpd.c

30 lines

Diff 172748

View Options

usr.sbin/lpr/lpd/lpd.8