Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/external/mpl/dhcp/dist merge local changes between dhcp-4.4....
details: https://anonhg.NetBSD.org/src/rev/db05fadf078f
branches: trunk
changeset: 1021328:db05fadf078f
user: christos <christos%NetBSD.org@localhost>
date: Wed May 26 22:52:31 2021 +0000
description:
merge local changes between dhcp-4.4.2 and dhcp-4.4.2-P1
diffstat:
external/mpl/dhcp/dist/client/dhclient.c | 12 +-
external/mpl/dhcp/dist/common/parse.c | 11 +-
external/mpl/dhcp/dist/common/tests/option_unittest.c | 88 ++++++++++++++++++-
external/mpl/dhcp/dist/relay/dhcrelay.c | 19 +--
external/mpl/dhcp/dist/relay/tests/relay_unittests.c | 30 ++++++-
external/mpl/dhcp/dist/server/dhcpd.c | 10 +-
6 files changed, 140 insertions(+), 30 deletions(-)
diffs (truncated from 380 to 300 lines):
diff -r 905e83147e46 -r db05fadf078f external/mpl/dhcp/dist/client/dhclient.c
--- a/external/mpl/dhcp/dist/client/dhclient.c Wed May 26 22:48:40 2021 +0000
+++ b/external/mpl/dhcp/dist/client/dhclient.c Wed May 26 22:52:31 2021 +0000
@@ -1,11 +1,11 @@
-/* $NetBSD: dhclient.c,v 1.3 2020/08/03 21:10:56 christos Exp $ */
+/* $NetBSD: dhclient.c,v 1.4 2021/05/26 22:52:31 christos Exp $ */
/* dhclient.c
DHCP Client. */
/*
- * Copyright (c) 2004-2020 by Internet Systems Consortium, Inc. ("ISC")
+ * Copyright (c) 2004-2021 by Internet Systems Consortium, Inc. ("ISC")
* Copyright (c) 1995-2003 by Internet Software Consortium
*
* This Source Code Form is subject to the terms of the Mozilla Public
@@ -33,7 +33,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: dhclient.c,v 1.3 2020/08/03 21:10:56 christos Exp $");
+__RCSID("$NetBSD: dhclient.c,v 1.4 2021/05/26 22:52:31 christos Exp $");
#include "dhcpd.h"
#include <isc/util.h>
@@ -84,7 +84,7 @@
#define ASSERT_STATE(state_is, state_shouldbe) {}
#ifndef UNIT_TEST
-static const char copyright[] = "Copyright 2004-2020 Internet Systems Consortium.";
+static const char copyright[] = "Copyright 2004-2021 Internet Systems Consortium.";
static const char arr [] = "All rights reserved.";
static const char message [] = "Internet Systems Consortium DHCP Client";
static const char url [] = "For info, please visit https://www.isc.org/software/dhcp/";;
@@ -241,7 +241,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: dhclient.c,v 1.3 2020/08/03 21:10:56 christos Exp $");
+__RCSID("$NetBSD: dhclient.c,v 1.4 2021/05/26 22:52:31 christos Exp $");
#if defined(DHCPv6) && defined(DHCP4o6)
static void dhcp4o6_poll(void *dummy);
@@ -1230,7 +1230,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: dhclient.c,v 1.3 2020/08/03 21:10:56 christos Exp $");
+__RCSID("$NetBSD: dhclient.c,v 1.4 2021/05/26 22:52:31 christos Exp $");
void state_reboot (cpp)
void *cpp;
diff -r 905e83147e46 -r db05fadf078f external/mpl/dhcp/dist/common/parse.c
--- a/external/mpl/dhcp/dist/common/parse.c Wed May 26 22:48:40 2021 +0000
+++ b/external/mpl/dhcp/dist/common/parse.c Wed May 26 22:52:31 2021 +0000
@@ -1,11 +1,11 @@
-/* $NetBSD: parse.c,v 1.3 2020/08/03 21:10:56 christos Exp $ */
+/* $NetBSD: parse.c,v 1.4 2021/05/26 22:52:31 christos Exp $ */
/* parse.c
Common parser code for dhcpd and dhclient. */
/*
- * Copyright (c) 2004-2019 by Internet Systems Consortium, Inc. ("ISC")
+ * Copyright (c) 2004-2021 by Internet Systems Consortium, Inc. ("ISC")
* Copyright (c) 1995-2003 by Internet Software Consortium
*
* This Source Code Form is subject to the terms of the Mozilla Public
@@ -29,7 +29,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: parse.c,v 1.3 2020/08/03 21:10:56 christos Exp $");
+__RCSID("$NetBSD: parse.c,v 1.4 2021/05/26 22:52:31 christos Exp $");
#include "dhcpd.h"
#include <isc/util.h>
@@ -5561,13 +5561,14 @@
skip_to_semi (cfile);
return 0;
}
- convert_num (cfile, &buf [len], val, 16, 8);
- if (len++ > max) {
+ if (len >= max) {
parse_warn (cfile,
"hexadecimal constant too long.");
skip_to_semi (cfile);
return 0;
}
+ convert_num (cfile, &buf [len], val, 16, 8);
+ len++;
token = peek_token (&val, (unsigned *)0, cfile);
if (token == COLON)
token = next_token (&val,
diff -r 905e83147e46 -r db05fadf078f external/mpl/dhcp/dist/common/tests/option_unittest.c
--- a/external/mpl/dhcp/dist/common/tests/option_unittest.c Wed May 26 22:48:40 2021 +0000
+++ b/external/mpl/dhcp/dist/common/tests/option_unittest.c Wed May 26 22:52:31 2021 +0000
@@ -1,7 +1,7 @@
-/* $NetBSD: option_unittest.c,v 1.2 2018/04/07 22:37:29 christos Exp $ */
+/* $NetBSD: option_unittest.c,v 1.3 2021/05/26 22:52:31 christos Exp $ */
/*
- * Copyright (C) 2018 Internet Systems Consortium, Inc. ("ISC")
+ * Copyright (C) 2018-2021 Internet Systems Consortium, Inc. ("ISC")
*
* This Source Code Form is subject to the terms of the Mozilla Public
* License, v. 2.0. If a copy of the MPL was not distributed with this
@@ -131,6 +131,89 @@
}
}
+ATF_TC(parse_X);
+
+ATF_TC_HEAD(parse_X, tc)
+{
+ atf_tc_set_md_var(tc, "descr",
+ "Verify parse_X survices option too big.");
+}
+
+/* Initializes a parse struct from an input buffer of data. */
+static void init_parse(struct parse *cfile, char* name, char *input) {
+ memset(cfile, 0, sizeof(struct parse));
+ cfile->tlname = name;
+ cfile->lpos = cfile->line = 1;
+ cfile->cur_line = cfile->line1;
+ cfile->prev_line = cfile->line2;
+ cfile->token_line = cfile->cur_line;
+ cfile->cur_line[0] = cfile->prev_line[0] = 0;
+ cfile->file = -1;
+ cfile->eol_token = 0;
+
+ cfile->inbuf = input;
+ cfile->buflen = strlen(input);
+ cfile->bufsiz = 0;
+}
+
+/*
+ * This test verifies that parse_X does not overwrite the output
+ * buffer when given input data that exceeds the output buffer
+ * capacity.
+*/
+ATF_TC_BODY(parse_X, tc)
+{
+ struct parse cfile;
+ u_int8_t output[10];
+ unsigned len;
+
+ /* Input hex literal */
+ char *input = "01:02:03:04:05:06:07:08";
+ unsigned expected_len = 8;
+
+ /* Normal output plus two filler bytes */
+ u_int8_t expected_plus_two[] = {
+ 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0xff, 0xff
+ };
+
+ /* Safe output when option is too long */
+ unsigned short_buf_len = 4;
+ u_int8_t expected_too_long[] = {
+ 0x01, 0x02, 0x03, 0x04, 0xff, 0xff, 0xff, 0xff, 0xff, 0xff
+ };
+
+ /* First we'll run one that works normally */
+ memset(output, 0xff, sizeof(output));
+ init_parse(&cfile, "hex_fits", input);
+
+ len = parse_X(&cfile, output, expected_len);
+
+ // Len should match the expected len.
+ if (len != expected_len) {
+ atf_tc_fail("parse_X failed, output len: %d", len);
+ }
+
+ // We should not have written anything past the end of the buffer.
+ if (memcmp(output, expected_plus_two, sizeof(output))) {
+ atf_tc_fail("parse_X failed, output does not match expected");
+ }
+
+ // Now we'll try it with a buffer that's too small.
+ init_parse(&cfile, "hex_too_long", input);
+ memset(output, 0xff, sizeof(output));
+
+ len = parse_X(&cfile, output, short_buf_len);
+
+ // On errors, len should be zero.
+ if (len != 0) {
+ atf_tc_fail("parse_X failed, we should have had an error");
+ }
+
+ // We should not have written anything past the end of the buffer.
+ if (memcmp(output, expected_too_long, sizeof(output))) {
+ atf_tc_fail("parse_X overwrote buffer!");
+ }
+}
/* This macro defines main() method that will call specified
test cases. tp and simple_test_case names can be whatever you want
@@ -139,6 +222,7 @@
{
ATF_TP_ADD_TC(tp, option_refcnt);
ATF_TP_ADD_TC(tp, pretty_print_option);
+ ATF_TP_ADD_TC(tp, parse_X);
return (atf_no_error());
}
diff -r 905e83147e46 -r db05fadf078f external/mpl/dhcp/dist/relay/dhcrelay.c
--- a/external/mpl/dhcp/dist/relay/dhcrelay.c Wed May 26 22:48:40 2021 +0000
+++ b/external/mpl/dhcp/dist/relay/dhcrelay.c Wed May 26 22:52:31 2021 +0000
@@ -1,11 +1,11 @@
-/* $NetBSD: dhcrelay.c,v 1.4 2021/05/21 21:07:37 christos Exp $ */
+/* $NetBSD: dhcrelay.c,v 1.5 2021/05/26 22:52:32 christos Exp $ */
/* dhcrelay.c
DHCP/BOOTP Relay Agent. */
/*
- * Copyright(c) 2004-2020 by Internet Systems Consortium, Inc.("ISC")
+ * Copyright(c) 2004-2021 by Internet Systems Consortium, Inc.("ISC")
* Copyright(c) 1997-2003 by Internet Software Consortium
*
* This Source Code Form is subject to the terms of the Mozilla Public
@@ -29,7 +29,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: dhcrelay.c,v 1.4 2021/05/21 21:07:37 christos Exp $");
+__RCSID("$NetBSD: dhcrelay.c,v 1.5 2021/05/26 22:52:32 christos Exp $");
#include "dhcpd.h"
#include <syslog.h>
@@ -169,7 +169,7 @@
static void request_v4_interface(const char* name, int flags);
static const char copyright[] =
-"Copyright 2004-2020 Internet Systems Consortium.";
+"Copyright 2004-2021 Internet Systems Consortium.";
static const char arr[] = "All rights reserved.";
static const char message[] =
"Internet Systems Consortium DHCP Relay Agent";
@@ -264,7 +264,7 @@
*/
#include <sys/cdefs.h>
-__RCSID("$NetBSD: dhcrelay.c,v 1.4 2021/05/21 21:07:37 christos Exp $");
+__RCSID("$NetBSD: dhcrelay.c,v 1.5 2021/05/26 22:52:32 christos Exp $");
static const char use_noarg[] = "No argument for command: %s";
#ifdef RELAY_PORT
static const char use_port_defined[] = "Port already set, %s inappropriate";
@@ -1465,8 +1465,9 @@
return (length);
}
+#ifndef UNIT_TEST
+
#ifdef DHCPv6
-#ifndef UNIT_TEST
/*
* Parse a downstream argument: [address%]interface[#index].
*/
@@ -2051,14 +2052,12 @@
if (if_id.data != NULL)
data_string_forget(&if_id, MDL);
}
-#endif /* UNIT_TEST */
/*
* Called by the dispatch packet handler with a decoded packet.
*/
void
dhcpv6(struct packet *packet) {
-#ifndef UNIT_TEST
struct stream_list *dp;
/* Try all relay-replies downwards. */
@@ -2081,9 +2080,8 @@
log_info("Can't process packet from interface '%s'.",
packet->interface->name);
-#endif /* UNIT_TEST */
}
-#endif /* DHCPv6 */
+#endif
/* Stub routines needed for linking with DHCP libraries. */
void
@@ -2181,3 +2179,4 @@
interface_snorf(tmp, (INTERFACE_REQUESTED | flags));
interface_dereference(&tmp, MDL);
}
+#endif /* UNIT_TEST */
diff -r 905e83147e46 -r db05fadf078f external/mpl/dhcp/dist/relay/tests/relay_unittests.c
--- a/external/mpl/dhcp/dist/relay/tests/relay_unittests.c Wed May 26 22:48:40 2021 +0000
+++ b/external/mpl/dhcp/dist/relay/tests/relay_unittests.c Wed May 26 22:52:31 2021 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: relay_unittests.c,v 1.2 2020/08/03 21:10:57 christos Exp $ */
+/* $NetBSD: relay_unittests.c,v 1.3 2021/05/26 22:52:32 christos Exp $ */
Home |
Main Index |
Thread Index |
Old Index