Age | Commit message (Collapse) | Author |
|
|
|
|
|
Fix a regression introduced in previous.
|
|
matches the corresponding RTM_GET request.
Based on a submission from Julien Dhaille, ok bluhm@
|
|
TOK_* case:'s alphabetically to make it easier to
find things.
|
|
returning 0/1 for success, etc. Simplify it
by only supporting a single ip address per
statement, as dhclient.conf(5) states. Don't
add duplicate addresses to the reject list.
|
|
|
|
to parse_hex_octets() and have it return 0/1 to indicate success,
to not touch existing data on error paths, to handle ';'
better, emit single error message ("expecting colon
delimited list of hex octets."). Fold parse_hex() into
parse_hex_octets().
Simplify logic in parse_option_decl() to eliminate duplicate
overflow error checking and error messages as a result.
|
|
to not touch existing data on error paths, to handle ';'
better, emit single error message ("expecting comma
delimited list of option names.").
|
|
to not touch existing data on error paths, to handle ';'
better.
|
|
|
|
|
|
0 for failure, 1 for success, emitting a
single error message ("expecting string.") and
handling ';' better.
Don't leak memory when encountering pathological
config or lease files containing repeated instances
of an option or command with string data.
|
|
committed.
|
|
some 'return;'s that should be 'break;'s.
|
|
code in the same way as the other parse_*()
functions. Whitespace fix in comment while
passing. Also conform to idiom that original
data is not changed in the error path.
|
|
single error message ("expecting integer between x and y")
and to properly handle ';' in error cases.
|
|
file. Broken due to incorrect translation from a netmask
to the count of leading 1's. Use brute force counting
until something better is found.
Issue discovered and diffs tested by Anthony Coulter via
bugs@. Thanks!
|
|
i.e. handle ';' better, and issue error
message ("expecting boolean.") itself.
|
|
i.e. handle ';' better, and issue only one error
message ("expecting IPv4 CIDR block").
One gratuitous whitespace tweak tossed in.
|
|
failure ("expecting unsigned 32-bit decimal value") and to
properly handle the terminating ';' in error situations.
Make parse_lease_time() return an int to indicate success
or failure as its friends do. Also avoid swapping endianess
twice.
Use == 1 vs != 0 when checking parse_ip_addr() return
value.
|
|
failure ("expecting IPv4 address") and to properly
handle the terminating ';' in error situations.
|
|
pointlessly precise error messages in favour of 'expecting
UTC time'.
|
|
|
|
dhclient.conf.
|
|
|
|
set to a non-zero value in dhclient.conf(5).
Fixes the bsd.rd upgrade issue reported by
Eivinde Eide via misc@.
|
|
OK mikeb@
|
|
ignore offers containing unknown DHCP
options. Broken in 2013 and nobody
noticed.
ok mpi@
|
|
around packet -> lease processing.
|
|
DHCP options). Broken by r1.204, 4 years, 8 months
ago.
|
|
use standard error messages for malloc errors.
|
|
|
|
name of the function that failed and the significant
parameters. Distinguish between poll() errors and
problematic revents values.
|
|
|
|
|
|
|
|
providing the name of the packet type that causes
an error.
|
|
|
|
if send_packet(DISCOVER) fails.
Distinguish between writev() and sendmsg() errors in
send_packet() log messages.
Check for short writes no matter how the packet is
output. Return failure in this occurs. Check results
of all send_packet() calls.
If send_packet() fails, don't log that the DHCP message
was sent.
|
|
need to complicate things by passing it a string
identifying where it was called. Tweak log
verbiage.
|
|
`finalize' function with the `fail' argument when this happen.
Introduce some sanity checks in exchange_free() to be able to call if
even if the data structure isn't completely initialized.
Plug memory leaks when exchange_establish() fails. While here fix a
double free in one of the error paths.
Based on a diff from hshoexer@, ok stsp@, markus@
|
|
|
|
Create global 'log_procname' and set it to '<ifname>' or
'<ifname> [priv]' as appropriate for the process doing the
setting. Use it as the prefix in all log_*() output. Makes
tracking messages for an interface or a process much easier.
|
|
'<ifname> [priv]' as appropriate for the process doing the
setting. Use it as the prefix in all log_*() output. Makes
tracking messages for an interface or a process much easier.
|
|
a routing socket. Fixes at least one cause of resolv.conf
confusion and possibly hanging/looping dhclient if the
RTM_GET gets lost.
Fingered by phessler@ when doing many suspend/resumes
while switching between wifi and wired interfaces.
Testing & ok phessler@
|
|
both early and after forking. Makes fatal[x]() messages more
informative by always including the relevant interface name. And
"[proc]" in privileged process messages.
Simplify some logic by avoiding error message construction that is
no longer required.
|
|
ok/feedback sthen@ jmc@
|
|
|
|
verbiage.
|