dhclient: allow to supersede interface-mtu option
ClosedPublic
Actions

Authored by novel on May 18 2018, 7:22 PM.

Details

Reviewers

cem
wblock

Group Reviewers

manpages

Commits

rS334789: MFC: r334443 (by cem@) MF stable/11: r334787
rS334787: MFC: r334443 (by cem@)
rS334443: dhclient(8): allow to supersede interface-mtu option

Summary

In some cases broken DHCP servers might send invalid MTU value,
so allow to use 'supersede' in dhclient.conf to override this. When
superseded value is 0, MTU value is not updated at all.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

novel created this revision.May 18 2018, 7:22 PM

Herald added a reviewer: manpages. · View Herald TranscriptMay 18 2018, 7:22 PM

Herald added a subscriber: imp. · View Herald Transcript

Harbormaster completed remote builds in B16677: Diff 42708.May 18 2018, 7:22 PM

cem accepted this revision.May 18 2018, 8:58 PM

cem added inline comments.

dhclient/dhclient.c
839–841 ↗	(On Diff #42708)	Maybe we should still warn if the server sends zero, vs the user configuring it. But I don't feel strongly about it.
dhclient/dhclient.conf.5
234–235 ↗	(On Diff #42708)	It doesn't seem clear to me from the text that the special zero value only applies if supersede is specified; also it seems a little weird to put it in the "OPTION MODIFIERS" section. I'd suggest: .Pp Some options have special values: .Bl -tag -width indent .It Ar interface-mtu Any server-supplied interface MTU is ignored by the client if a .Ic supersede zero value is configured. But maybe get someone from docs to offer advice.

This revision is now accepted and ready to land.May 18 2018, 8:58 PM

novel updated this revision to Diff 42730.May 19 2018, 6:08 AM

novel added a reviewer: wblock.

This revision now requires review to proceed.May 19 2018, 6:08 AM

Harbormaster completed remote builds in B16693: Diff 42730.May 19 2018, 6:08 AM

cem accepted this revision.May 19 2018, 6:11 AM

cem added inline comments.

dhclient/dhclient.c
843 ↗	(On Diff #42730)	All of these parentheses seem excessive -- just `if (!supersede \|\| mtu == 0)` would do.

This revision is now accepted and ready to land.May 19 2018, 6:11 AM

@cem I'd appreciate if you could get this committed for me. I guess it needs to be properly MFCed so it gets into 11.2, and given that timeline is tight and I'm not very familiar with the process, it'll be better to get it done right from the first attempt.

dhclient/dhclient.c
843 ↗	(On Diff #42730)	This way it'll still print a warning for superseded value because 'mtu == 0' will be true.

novel updated this revision to Diff 42731.May 19 2018, 6:35 AM

This revision now requires review to proceed.May 19 2018, 6:35 AM

Harbormaster completed remote builds in B16694: Diff 42731.May 19 2018, 6:35 AM

In D15484#326885, @novel wrote:

@cem I'd appreciate if you could get this committed for me.

Sure, no problem. I'd like to see some manpages/doc team feedback; hopefully that happens soon.

I guess it needs to be properly MFCed so it gets into 11.2, and given that timeline is tight and I'm not very familiar with the process, it'll be better to get it done right from the first attempt.

I don't MFC, but maybe someone can be recruited to do it.