469 lines
14 KiB
Diff
469 lines
14 KiB
Diff
From a26161b0fd45cdbeed3038ac63ff04e3b727248f Mon Sep 17 00:00:00 2001
|
|
From: Pavel Zhukov <pzhukov@redhat.com>
|
|
Date: Thu, 21 Feb 2019 10:19:47 +0100
|
|
Subject: [PATCH 02/26] additional dhclient options
|
|
Cc: pzhukov@redhat.com
|
|
|
|
---
|
|
client/clparse.c | 10 +-
|
|
client/dhclient.8 | 27 ++++++
|
|
client/dhclient.c | 271 +++++++++++++++++++++++++++++++++++++++++++++++++++-
|
|
common/conflex.c | 2 +
|
|
includes/dhcpd.h | 3 +
|
|
includes/dhctoken.h | 1 +
|
|
6 files changed, 308 insertions(+), 6 deletions(-)
|
|
|
|
diff --git a/client/clparse.c b/client/clparse.c
|
|
index eaf48a8..7212e3a 100644
|
|
--- a/client/clparse.c
|
|
+++ b/client/clparse.c
|
|
@@ -189,6 +189,7 @@ isc_result_t read_client_conf ()
|
|
/* Requested lease time, used by DHCPv6 (DHCPv4 uses the option cache)
|
|
*/
|
|
top_level_config.requested_lease = 7200;
|
|
+ top_level_config.bootp_broadcast_always = 0;
|
|
|
|
group_allocate (&top_level_config.on_receipt, MDL);
|
|
if (!top_level_config.on_receipt)
|
|
@@ -394,7 +395,8 @@ void read_client_leases ()
|
|
interface-declaration |
|
|
LEASE client-lease-statement |
|
|
ALIAS client-lease-statement |
|
|
- KEY key-definition */
|
|
+ KEY key-definition |
|
|
+ BOOTP_BROADCAST_ALWAYS */
|
|
|
|
void parse_client_statement (cfile, ip, config)
|
|
struct parse *cfile;
|
|
@@ -817,6 +819,12 @@ void parse_client_statement (cfile, ip, config)
|
|
parse_lease_id_format(cfile);
|
|
break;
|
|
|
|
+ case BOOTP_BROADCAST_ALWAYS:
|
|
+ token = next_token(&val, (unsigned*)0, cfile);
|
|
+ config -> bootp_broadcast_always = 1;
|
|
+ parse_semi (cfile);
|
|
+ return;
|
|
+
|
|
|
|
default:
|
|
lose = 0;
|
|
diff --git a/client/dhclient.8 b/client/dhclient.8
|
|
index ebc750f..6d7fbdb 100644
|
|
--- a/client/dhclient.8
|
|
+++ b/client/dhclient.8
|
|
@@ -134,6 +134,33 @@ dhclient - Dynamic Host Configuration Protocol Client
|
|
.B -w
|
|
]
|
|
[
|
|
+.B -B
|
|
+]
|
|
+[
|
|
+.B -C
|
|
+.I dhcp-client-identifier
|
|
+]
|
|
+[
|
|
+.B -H
|
|
+.I host-name
|
|
+]
|
|
+[
|
|
+.B -F
|
|
+.I fqdn.fqdn
|
|
+]
|
|
+[
|
|
+.B -V
|
|
+.I vendor-class-identifier
|
|
+]
|
|
+[
|
|
+.B --request-options
|
|
+.I request-option-list
|
|
+]
|
|
+[
|
|
+.B --timeout
|
|
+.I timeout
|
|
+]
|
|
+[
|
|
.B --dad-wait-time
|
|
.I seconds
|
|
]
|
|
diff --git a/client/dhclient.c b/client/dhclient.c
|
|
index 825ab00..26a333c 100644
|
|
--- a/client/dhclient.c
|
|
+++ b/client/dhclient.c
|
|
@@ -41,6 +41,12 @@
|
|
#include <sys/wait.h>
|
|
#include <limits.h>
|
|
|
|
+/*
|
|
+ * Defined in stdio.h when _GNU_SOURCE is set, but we don't want to define
|
|
+ * that when building ISC code.
|
|
+ */
|
|
+extern int asprintf(char **strp, const char *fmt, ...);
|
|
+
|
|
TIME default_lease_time = 43200; /* 12 hours... */
|
|
TIME max_lease_time = 86400; /* 24 hours... */
|
|
|
|
@@ -110,6 +116,10 @@ char *mockup_relay = NULL;
|
|
|
|
char *progname = NULL;
|
|
|
|
+int bootp_broadcast_always = 0;
|
|
+
|
|
+extern struct option *default_requested_options[];
|
|
+
|
|
void run_stateless(int exit_mode, u_int16_t port);
|
|
|
|
static isc_result_t write_duid(struct data_string *duid);
|
|
@@ -183,8 +193,12 @@ static const char use_v6command[] = "Command not used for DHCPv4: %s";
|
|
" [-s server-addr] [-cf config-file]\n" \
|
|
" [-df duid-file] [-lf lease-file]\n" \
|
|
" [-pf pid-file] [--no-pid] [-e VAR=val]\n" \
|
|
-" [-sf script-file] [interface]*"
|
|
-
|
|
+" [-sf script-file] [interface]*\n" \
|
|
+" [-C <dhcp-client-identifier>] [-B]\n" \
|
|
+" [-H <host-name> | -F <fqdn.fqdn>] [--timeout <timeout>]\n" \
|
|
+" [-V <vendor-class-identifier>]\n" \
|
|
+" [--request-options <request option list>]"
|
|
+
|
|
#define DHCLIENT_USAGEH "{--version|--help|-h}"
|
|
|
|
static void
|
|
@@ -243,6 +257,16 @@ main(int argc, char **argv) {
|
|
#else
|
|
progname = argv[0];
|
|
#endif
|
|
+ char *dhcp_client_identifier_arg = NULL;
|
|
+ char *dhcp_host_name_arg = NULL;
|
|
+ char *dhcp_fqdn_arg = NULL;
|
|
+ char *dhcp_vendor_class_identifier_arg = NULL;
|
|
+ char *dhclient_request_options = NULL;
|
|
+
|
|
+ int timeout_arg = 0;
|
|
+ char *arg_conf = NULL;
|
|
+ int arg_conf_len = 0;
|
|
+
|
|
/* Initialize client globals. */
|
|
memset(&default_duid, 0, sizeof(default_duid));
|
|
|
|
@@ -558,6 +582,89 @@ main(int argc, char **argv) {
|
|
std_dhcid = 1;
|
|
} else if (!strcmp(argv[i], "-v")) {
|
|
quiet = 0;
|
|
+ } else if (!strcmp(argv[i], "-C")) {
|
|
+ if ((++i == argc) || (argv[i] == NULL) || (*(argv[i])=='\0')) {
|
|
+ usage(use_noarg, argv[i-1]);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (strlen(argv[i]) >= DHCP_MAX_OPTION_LEN) {
|
|
+ log_error("-C option dhcp-client-identifier string \"%s\" is too long - maximum length is: %d", argv[i], DHCP_MAX_OPTION_LEN-1);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ dhcp_client_identifier_arg = argv[i];
|
|
+ } else if (!strcmp(argv[i], "-B")) {
|
|
+ bootp_broadcast_always = 1;
|
|
+ } else if (!strcmp(argv[i], "-H")) {
|
|
+ if ((++i == argc) || (argv[i] == NULL) || (*(argv[i])=='\0')) {
|
|
+ usage(use_noarg, argv[i-1]);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (strlen(argv[i]) >= DHCP_MAX_OPTION_LEN) {
|
|
+ log_error("-H option host-name string \"%s\" is too long - maximum length is: %d", argv[i], DHCP_MAX_OPTION_LEN-1);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (dhcp_host_name_arg != NULL) {
|
|
+ log_error("The -H <host-name> and -F <fqdn> arguments are mutually exclusive");
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ dhcp_host_name_arg = argv[i];
|
|
+ } else if (!strcmp(argv[i], "-F")) {
|
|
+ if ((++i == argc) || (argv[i] == NULL) || (*(argv[i])=='\0')) {
|
|
+ usage(use_noarg, argv[i-1]);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (strlen(argv[i]) >= DHCP_MAX_OPTION_LEN) {
|
|
+ log_error("-F option fqdn.fqdn string \"%s\" is too long - maximum length is: %d", argv[i], DHCP_MAX_OPTION_LEN-1);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (dhcp_fqdn_arg != NULL) {
|
|
+ log_error("Only one -F <fqdn> argument can be specified");
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (dhcp_host_name_arg != NULL) {
|
|
+ log_error("The -F <fqdn> and -H <host-name> arguments are mutually exclusive");
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ dhcp_fqdn_arg = argv[i];
|
|
+ } else if (!strcmp(argv[i], "--timeout")) {
|
|
+ if ((++i == argc) || (argv[i] == NULL) || (*(argv[i])=='\0')) {
|
|
+ usage(use_noarg, argv[i-1]);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if ((timeout_arg = atoi(argv[i])) <= 0) {
|
|
+ log_error("timeout option must be > 0 - bad value: %s",argv[i]);
|
|
+ exit(1);
|
|
+ }
|
|
+ } else if (!strcmp(argv[i], "-V")) {
|
|
+ if ((++i == argc) || (argv[i] == NULL) || (*(argv[i])=='\0')) {
|
|
+ usage(use_noarg, argv[i-1]);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ if (strlen(argv[i]) >= DHCP_MAX_OPTION_LEN) {
|
|
+ log_error("-V option vendor-class-identifier string \"%s\" is too long - maximum length is: %d", argv[i], DHCP_MAX_OPTION_LEN-1);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ dhcp_vendor_class_identifier_arg = argv[i];
|
|
+ } else if (!strcmp(argv[i], "--request-options")) {
|
|
+ if ((++i == argc) || (argv[i] == NULL) || (*(argv[i])=='\0')) {
|
|
+ usage(use_noarg, argv[i-1]);
|
|
+ exit(1);
|
|
+ }
|
|
+
|
|
+ dhclient_request_options = argv[i];
|
|
+
|
|
} else if (argv[i][0] == '-') {
|
|
usage("Unknown command: %s", argv[i]);
|
|
} else if (interfaces_requested < 0) {
|
|
@@ -754,6 +861,156 @@ main(int argc, char **argv) {
|
|
/* Parse the dhclient.conf file. */
|
|
read_client_conf();
|
|
|
|
+ /* Parse any extra command line configuration arguments: */
|
|
+ if ((dhcp_client_identifier_arg != NULL) && (*dhcp_client_identifier_arg != '\0')) {
|
|
+ arg_conf_len = asprintf(&arg_conf, "send dhcp-client-identifier \"%s\";", dhcp_client_identifier_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -C option dhcp-client-identifier");
|
|
+ }
|
|
+
|
|
+ if ((dhcp_host_name_arg != NULL) && (*dhcp_host_name_arg != '\0')) {
|
|
+ if (arg_conf == 0) {
|
|
+ arg_conf_len = asprintf(&arg_conf, "send host-name \"%s\";", dhcp_host_name_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -H option host-name");
|
|
+ } else {
|
|
+ char *last_arg_conf = arg_conf;
|
|
+ arg_conf = NULL;
|
|
+ arg_conf_len = asprintf(&arg_conf, "%s\nsend host-name \"%s\";", last_arg_conf, dhcp_host_name_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -H option host-name");
|
|
+
|
|
+ free(last_arg_conf);
|
|
+ }
|
|
+ }
|
|
+
|
|
+ if ((dhcp_fqdn_arg != NULL) && (*dhcp_fqdn_arg != '\0')) {
|
|
+ if (arg_conf == 0) {
|
|
+ arg_conf_len = asprintf(&arg_conf, "send fqdn.fqdn \"%s\";", dhcp_fqdn_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -F option fqdn.fqdn");
|
|
+ } else {
|
|
+ char *last_arg_conf = arg_conf;
|
|
+ arg_conf = NULL;
|
|
+ arg_conf_len = asprintf(&arg_conf, "%s\nsend fqdn.fqdn \"%s\";", last_arg_conf, dhcp_fqdn_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -F option fqdn.fqdn");
|
|
+
|
|
+ free(last_arg_conf);
|
|
+ }
|
|
+ }
|
|
+
|
|
+ if (timeout_arg) {
|
|
+ if (arg_conf == 0) {
|
|
+ arg_conf_len = asprintf(&arg_conf, "timeout %d;", timeout_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to process --timeout timeout argument");
|
|
+ } else {
|
|
+ char *last_arg_conf = arg_conf;
|
|
+ arg_conf = NULL;
|
|
+ arg_conf_len = asprintf(&arg_conf, "%s\ntimeout %d;", last_arg_conf, timeout_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len == 0))
|
|
+ log_fatal("Unable to process --timeout timeout argument");
|
|
+
|
|
+ free(last_arg_conf);
|
|
+ }
|
|
+ }
|
|
+
|
|
+ if ((dhcp_vendor_class_identifier_arg != NULL) && (*dhcp_vendor_class_identifier_arg != '\0')) {
|
|
+ if (arg_conf == 0) {
|
|
+ arg_conf_len = asprintf(&arg_conf, "send vendor-class-identifier \"%s\";", dhcp_vendor_class_identifier_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -V option vendor-class-identifier");
|
|
+ } else {
|
|
+ char *last_arg_conf = arg_conf;
|
|
+ arg_conf = NULL;
|
|
+ arg_conf_len = asprintf(&arg_conf, "%s\nsend vendor-class-identifier \"%s\";", last_arg_conf, dhcp_vendor_class_identifier_arg);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to send -V option vendor-class-identifier");
|
|
+
|
|
+ free(last_arg_conf);
|
|
+ }
|
|
+ }
|
|
+
|
|
+ if (dhclient_request_options != NULL) {
|
|
+ if (arg_conf == 0) {
|
|
+ arg_conf_len = asprintf(&arg_conf, "request %s;", dhclient_request_options);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to parse --request-options <request options list> argument");
|
|
+ } else {
|
|
+ char *last_arg_conf = arg_conf;
|
|
+ arg_conf = NULL;
|
|
+ arg_conf_len = asprintf(&arg_conf, "%s\nrequest %s;", last_arg_conf, dhclient_request_options);
|
|
+
|
|
+ if ((arg_conf == 0) || (arg_conf_len <= 0))
|
|
+ log_fatal("Unable to parse --request-options <request options list> argument");
|
|
+
|
|
+ free(last_arg_conf);
|
|
+ }
|
|
+ }
|
|
+
|
|
+ if (arg_conf) {
|
|
+ if (arg_conf_len == 0)
|
|
+ if ((arg_conf_len = strlen(arg_conf)) == 0)
|
|
+ /* huh ? cannot happen ! */
|
|
+ log_fatal("Unable to process -C/-H/-F/--timeout/-V/--request-options configuration arguments");
|
|
+
|
|
+ /* parse the extra dhclient.conf configuration arguments
|
|
+ * into top level config: */
|
|
+ struct parse *cfile = (struct parse *)0;
|
|
+ const char *val = NULL;
|
|
+ int token;
|
|
+
|
|
+ status = new_parse(&cfile, -1, arg_conf, arg_conf_len, "extra dhclient -C/-H/-F/--timeout/-V/--request-options configuration arguments", 0);
|
|
+
|
|
+ if ((status != ISC_R_SUCCESS) || (cfile -> warnings_occurred))
|
|
+ log_fatal("Cannot parse -C/-H/-F/--timeout/-V/--request-options configuration arguments !");
|
|
+ /* more detailed parse failures will be logged */
|
|
+
|
|
+ do {
|
|
+ token = peek_token(&val, (unsigned *)0, cfile);
|
|
+ if (token == END_OF_FILE)
|
|
+ break;
|
|
+
|
|
+ parse_client_statement(cfile, (struct interface_info *)0, &top_level_config);
|
|
+ } while (1);
|
|
+
|
|
+ if (cfile -> warnings_occurred)
|
|
+ log_fatal("Cannot parse -C/-H/-F/--timeout/-V/--request-options configuration arguments !");
|
|
+ end_parse(&cfile);
|
|
+
|
|
+ if (timeout_arg) {
|
|
+ /* we just set the toplevel timeout, but per-client
|
|
+ * timeouts may still be at defaults.
|
|
+ */
|
|
+ for (ip=interfaces; ip; ip = ip->next) {
|
|
+ if (ip->client->config->timeout == 60)
|
|
+ ip->client->config->timeout = timeout_arg;
|
|
+ }
|
|
+ }
|
|
+
|
|
+ if ((dhclient_request_options != 0) && (top_level_config.requested_options != default_requested_options)) {
|
|
+ for (ip=interfaces; ip; ip = ip->next) {
|
|
+ if (ip->client->config->requested_options == default_requested_options)
|
|
+ ip->client->config->requested_options = top_level_config.requested_options;
|
|
+ }
|
|
+ }
|
|
+
|
|
+ free(arg_conf);
|
|
+ arg_conf = NULL;
|
|
+ arg_conf_len = 0;
|
|
+ }
|
|
+
|
|
/* Parse the lease database. */
|
|
read_client_leases();
|
|
|
|
@@ -3226,7 +3483,8 @@ void make_discover (client, lease)
|
|
client -> packet.xid = random ();
|
|
client -> packet.secs = 0; /* filled in by send_discover. */
|
|
|
|
- if (can_receive_unicast_unconfigured (client -> interface))
|
|
+ if ((!(bootp_broadcast_always || client->config->bootp_broadcast_always))
|
|
+ && can_receive_unicast_unconfigured(client->interface))
|
|
client -> packet.flags = 0;
|
|
else
|
|
client -> packet.flags = htons (BOOTP_BROADCAST);
|
|
@@ -3311,7 +3569,9 @@ void make_request (client, lease)
|
|
} else {
|
|
memset (&client -> packet.ciaddr, 0,
|
|
sizeof client -> packet.ciaddr);
|
|
- if (can_receive_unicast_unconfigured (client -> interface))
|
|
+ if ((!(bootp_broadcast_always ||
|
|
+ client ->config->bootp_broadcast_always)) &&
|
|
+ can_receive_unicast_unconfigured (client -> interface))
|
|
client -> packet.flags = 0;
|
|
else
|
|
client -> packet.flags = htons (BOOTP_BROADCAST);
|
|
@@ -3374,7 +3634,8 @@ void make_decline (client, lease)
|
|
client -> packet.hops = 0;
|
|
client -> packet.xid = client -> xid;
|
|
client -> packet.secs = 0; /* Filled in by send_request. */
|
|
- if (can_receive_unicast_unconfigured (client -> interface))
|
|
+ if ((!(bootp_broadcast_always || client->config-> bootp_broadcast_always))
|
|
+ && can_receive_unicast_unconfigured (client->interface))
|
|
client -> packet.flags = 0;
|
|
else
|
|
client -> packet.flags = htons (BOOTP_BROADCAST);
|
|
diff --git a/common/conflex.c b/common/conflex.c
|
|
index 045b655..71c0bf5 100644
|
|
--- a/common/conflex.c
|
|
+++ b/common/conflex.c
|
|
@@ -832,6 +832,8 @@ intern(char *atom, enum dhcp_token dfv) {
|
|
if (!strcasecmp(atom+1, "ig-endian")) {
|
|
return TOKEN_BIG_ENDIAN;
|
|
}
|
|
+ if (!strcasecmp (atom + 1, "ootp-broadcast-always"))
|
|
+ return BOOTP_BROADCAST_ALWAYS;
|
|
break;
|
|
case 'c':
|
|
if (!strcasecmp(atom + 1, "ase"))
|
|
diff --git a/includes/dhcpd.h b/includes/dhcpd.h
|
|
index 5930e6a..018fa34 100644
|
|
--- a/includes/dhcpd.h
|
|
+++ b/includes/dhcpd.h
|
|
@@ -1269,6 +1269,9 @@ struct client_config {
|
|
|
|
int lease_id_format; /* format for IDs in lease file,
|
|
TOKEN_OCTAL or TOKEN_HEX */
|
|
+
|
|
+ int bootp_broadcast_always; /* If nonzero, always set the BOOTP_BROADCAST
|
|
+ flag in requests */
|
|
};
|
|
|
|
/* Per-interface state used in the dhcp client... */
|
|
diff --git a/includes/dhctoken.h b/includes/dhctoken.h
|
|
index 5920f4f..7e7215a 100644
|
|
--- a/includes/dhctoken.h
|
|
+++ b/includes/dhctoken.h
|
|
@@ -377,6 +377,7 @@ enum dhcp_token {
|
|
TOKEN_HEX = 677,
|
|
TOKEN_OCTAL = 678,
|
|
KEY_ALGORITHM = 679
|
|
+ BOOTP_BROADCAST_ALWAYS = 680
|
|
};
|
|
|
|
#define is_identifier(x) ((x) >= FIRST_TOKEN && \
|
|
--
|
|
2.14.5
|
|
|