ctld: Fix sign bug validating kernel ports when reloading config
ClosedPublic
Actions

Authored by jhb on Wed, Apr 2, 8:04 PM.

Details

Reviewers

Commits

rGe7611b7531c4: ctld: Fix sign bug validating kernel ports when reloading config

Summary

When I switched new_pports_from_conf to return bool instead of an
inverted int, I missed updating the call after reloading the config
file due to SIGHUP.

Fixes: 450a84c292ae ("ctld: Some bool-related cleanups")
Sponsored by: Chelsio Communications

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jhb created this revision.Wed, Apr 2, 8:04 PM

Herald added a subscriber: imp. · View Herald TranscriptWed, Apr 2, 8:04 PM

jhb requested review of this revision.Wed, Apr 2, 8:04 PM

Harbormaster completed remote builds in B63286: Diff 153040.Wed, Apr 2, 8:04 PM

jhb added a child revision: D49642: ctld: Reduce code duplication in auth_check_secret_length.Wed, Apr 2, 8:04 PM

This revision was not accepted when it landed; it landed in state Needs Review.Fri, Apr 11, 2:04 PM

Closed by commit rGe7611b7531c4: ctld: Fix sign bug validating kernel ports when reloading config (authored by jhb). · Explain Why

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGe7611b7531c4: ctld: Fix sign bug validating kernel ports when reloading config.

Revision Contents
Changeset List

Path

Size

usr.sbin/

ctld/

ctld.cc

2 lines

Diff 153517

View Options

ctld: Fix sign bug validating kernel ports when reloading configClosedPublicActions