| Age | Commit message (Collapse) | Author |
|---|
|
function prototypes.
|
|
Mkae 'ignore', 'request' and 'require' cumulative so all
options don't have to be jammed into one line.
|
|
behaviour.
Always go daemon after link_timeout seconds and complete lease
negotiations in the background if necessary. No hanging around in the
foreground for the full 64 seconds waiting for a server to appear.
Log a more relevant message when a default route can't be obtained via
RTM_GET. i.e. "no default route" rather than "No such process".
-q -> -v ok mpi@
|
|
function names and function code into sync.
|
|
|
|
in the names of the respective parsing functions.
|
|
parse_number('L') since it is parsing unsigned 32bit integers.
|
|
match grammar comments and improve euphony.
|
|
|
|
|
|
inside 'lease {}' declarations. We don't keep leases from
multiple interfaces in dhclient.leases.<if> files.
One less thing to parse and keep in memory. More rope for
those wanting to do unspeakable things with their lease files.
|
|
and move all lease parsing into parse_lease().
|
|
functions and fields to more obvious names reflecting this
separation.
|
|
|
|
statements. They are silly but harmless and work fine.
|
|
in struct client_config.
|
|
time, renewal time, rebinding time. Use the ACTION_DEFAULT mechanism
to set the default expiry time for any lease so gauche as to not
provide one. Use the DHCP default lease times for BOOTP leases instead
of, bizarrely, shorter times.
|
|
|
|
and into struct interface_info. Nuke set_lease_times().
|
|
renew/rebind/expiry. Treat renew/rebind/expiry statements in leases as
comments for human consumption.
|
|
received. Set it in packet_to_lease() and preserve it in
apply_defaults(). Otherwise not used, documented or printed in lease
database yet.
|
|
"do {} while (1)".
|
|
declarations and placing 'lease' declarations inside
'interface' declarations.
Document and enforce requirement that all 'lease'
declarations must specify the interface to which they
apply.
Ignore static leases that apply to other interfaces rather
than complaining the interface name is wrong and using the
lease anyway.
|
|
into the 0/1 for success world.
|
|
|
|
return 0/1 for success.
|
|
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.
|
|
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.
|
|
i.e. handle ';' better, and issue error
message ("expecting boolean.") itself.
|
|
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.
|
|
|
|
|
|
'<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.
|
|
verbiage.
|
|
No intentional functional change.
|
|
"<var> == 0", "!(<var> & FLAG)" with
"(<var> & FLAG) == 0", "!<func()>"
with "<func()> == 0" and "!<define>" with
"<define> == 0". And the positive cases
as well.
A few stray == NULL and != NULL as well.
|
|
"<pointer> == NULL". And of course "<pointer>"
expressions with "<pointer> != NULL".
|
|
declarations.
|
|
|
|
chars to parse_decimal() instead of less obvious *fmt; refactor
to eliminate need for the 'alloc:' and 'bad_flag:' labels and the
invidious backwards goto's to them.
|
