Page MenuHomeFreeBSD

ping: fix parsing of options including '4' and '6'

Authored by asomers on Oct 6 2021, 10:58 PM.



ping uses a two-pass option parser. The first pass determines whether
ipv4 or ipv6 is desired, and the second parses the rest of the options.
But the first pass wrongly detects a '4' or '6' in an option's value as
a request to use ipv6 or ipv6 respectively, for example in an invocation
like "ping -c6".

Fix this confusion by including all options in the first round of
parsing, but ignoring those unrelated to ipv4/ipv6 selection.

PR: 258048
Reported by:
Submitted by:
MFC after: 2 weeks

Test Plan

tests added

Diff Detail

R10 FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

should we add a test for exit:1 with -46 specified?


I wonder if we could cache this?

should we add a test for exit:1 with -46 specified?

Yeah, sure.


No, because every ATF test case runs in its own process. And it only takes about 5ms on my system, anyway.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 21 2021, 12:06 AM
This revision was automatically updated to reflect the committed changes.