From 71bdc6b873f87456bb6bcccb5058069a90191eea Mon Sep 17 00:00:00 2001 From: renmingshuai Date: Mon, 7 Feb 2022 22:06:15 +0800 Subject: [PATCH] fix coredump when client active is NULL --- dhcp.spec | 9 ++- fix-coredump-when-client-active-is-NULL.patch | 67 +++++++++++++++++++ 2 files changed, 75 insertions(+), 1 deletion(-) create mode 100644 fix-coredump-when-client-active-is-NULL.patch diff --git a/dhcp.spec b/dhcp.spec index abd3d6a..2dfcb71 100644 --- a/dhcp.spec +++ b/dhcp.spec @@ -3,7 +3,7 @@ Name: dhcp Version: 4.4.2 -Release: 6 +Release: 7 Summary: Dynamic host configuration protocol software #Please don't change the epoch on this package Epoch: 12 @@ -55,6 +55,7 @@ Patch32: bugfix-dhclient-check-if-pid-was-held.patch Patch33: bugfix-error-message-display.patch Patch34: bugfix-dhcp-64-bit-lease-parse.patch Patch35: CVE-2021-25217.patch +Patch36: fix-coredump-when-client-active-is-NULL.patch BuildRequires: gcc autoconf automake libtool openldap-devel krb5-devel libcap-ng-devel bind-export-devel BuildRequires: systemd systemd-devel @@ -296,6 +297,12 @@ exit 0 %{_mandir}/man3/omapi.3.gz %changelog +* Mon Feb 7 2022 renmingshuai - 4.4.2-7 +- Type:bugfix +- ID:NA +- SUG:restart +- DESC:fix coredump when client active is NULL + * Tue Sep 14 2021 panchenbo - 4.4.2-6 - DESC: install dhcpd.conf.example diff --git a/fix-coredump-when-client-active-is-NULL.patch b/fix-coredump-when-client-active-is-NULL.patch new file mode 100644 index 0000000..ff42768 --- /dev/null +++ b/fix-coredump-when-client-active-is-NULL.patch @@ -0,0 +1,67 @@ +From 098d7f8e2e2b468e6a615084fb4330fed49bb54a Mon Sep 17 00:00:00 2001 +From: hanzhijun +Date: Mon, 24 Aug 2020 15:39:53 +0800 +Subject: [PATCH] fix coredump when client -> active is NULL + +--- + client/dhclient.c | 37 +++++++++++++++++++++++++++++++++++++ + 1 file changed, 37 insertions(+) + +diff --git a/client/dhclient.c b/client/dhclient.c +index 2a17bfd..7581670 100644 +--- a/client/dhclient.c ++++ b/client/dhclient.c +@@ -2059,6 +2059,24 @@ void state_bound (cpp) + struct data_string ds; + + ASSERT_STATE(state, S_BOUND); ++ if (!client -> active) ++ { ++ struct timeval tv; ++ ++ log_error ("No active lease in bound."); ++ script_init(client, "FAIL", (struct string_list *)0); ++ if (client -> alias) ++ script_write_params(client, "alias_", client -> alias); ++ script_go(client); ++ client -> state = S_INIT; ++ tv.tv_sec = cur_tv.tv_sec + ((client->config->retry_interval + 1) / 2 + ++ (random() % client->config->retry_interval)); ++ tv.tv_usec = ((tv.tv_sec - cur_tv.tv_sec) > 1) ? ++ random() % 1000000 : cur_tv.tv_usec; ++ add_timeout(&tv, state_init, client, 0, 0); ++ detach (); ++ return; ++ } + + /* T1 has expired. */ + make_request (client, client -> active); +@@ -3173,6 +3191,25 @@ void send_request (cpp) + return; + } + ++ if (!client -> active && client -> state != S_REQUESTING) ++ { ++ struct timeval tv; ++ ++ log_error ("No active lease in request."); ++ script_init(client, "FAIL", (struct string_list *)0); ++ if (client -> alias) ++ script_write_params(client, "alias_", client -> alias); ++ script_go(client); ++ client -> state = S_INIT; ++ tv.tv_sec = cur_tv.tv_sec + ((client->config->retry_interval + 1) / 2 + ++ (random() % client->config->retry_interval)); ++ tv.tv_usec = ((tv.tv_sec - cur_tv.tv_sec) > 1) ? ++ random() % 1000000 : cur_tv.tv_usec; ++ add_timeout(&tv, state_init, client, 0, 0); ++ detach (); ++ return; ++ } ++ + /* If we're in the reboot state, make sure the media is set up + correctly. */ + if (client -> state == S_REBOOTING && +-- +2.27.0 + -- Gitee