Differential D56233

ip_carp: define CARP_AUTHLEN for carp_authlen field
ClosedPublic
Actions

Authored by chris.longros_gmail.com on Apr 3 2026, 2:19 PM.

Details

Reviewers

glebius

Summary

Replace the bare 7 with a named constant as noted by the
existing XXX DEFINE comment.

Signed-off-by: Christos Longros <chris.longros@gmail.com>

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 71934
Build 68817: arc lint + arc unit

Event Timeline

chris.longros_gmail.com created this revision.Apr 3 2026, 2:19 PM

Herald added subscribers: glebius, melifaro, imp. · View Herald TranscriptApr 3 2026, 2:19 PM

chris.longros_gmail.com requested review of this revision.Apr 3 2026, 2:19 PM

Harbormaster completed remote builds in B71934: Diff 174857.Apr 3 2026, 2:19 PM

chris.longros_gmail.com added a reviewer: glebius.Apr 3 2026, 2:20 PM

Thanks for the change!

I'm leaving for a short offline vacation and I prefer not to push changes before that. If you find an src committer who can push - welcome. Otherwise I will be back in about a week.

This revision is now accepted and ready to land.Apr 3 2026, 3:31 PM

In D56233#1286789, @glebius wrote:

Thanks for the change!

I'm leaving for a short offline vacation and I prefer not to push changes before that. If you find an src committer who can push - welcome. Otherwise I will be back in about a week.

That's fine. The commit can wait 1 week :) Have a good time.

@glebius Hi, could you land this at your convenience? Thanks!

In D56233#1286789, @glebius wrote:

Thanks for the change!

I'm leaving for a short offline vacation and I prefer not to push changes before that. If you find an src committer who can push - welcome. Otherwise I will be back in about a week.

Any chance of committing this to main ?
Thanks :)