diff options
| author | Anatole Denis <natolumin@unverle.fr> | 2019-10-08 14:02:04 +0200 |
|---|---|---|
| committer | Chris K <c@chrisko.ch> | 2020-06-20 21:42:12 -0700 |
| commit | ff7ce1d50285d8f5467dcb4b8dd4dc08b8fc1f79 (patch) | |
| tree | d4f5be625f7376fbd5e2b5cb9ae448781db4e335 /dhcpv4/dhcpv4_test.go | |
| parent | 65668712da12ab11520b64097496616d39a50368 (diff) | |
dhcpv4: Avoid a panic in ToBytes() with long strings
When BootFileName is longer than 128 bytes or ServerHostName is longer
than 64 bytes, trying to null-terminate the strings when writing out the
packet causes a panic.
Since the ToBytes() function cannot return errors, silently truncate the
string instead (we do the same with ClientHWAddr if it is longer than 16
bytes for example)
Signed-off-by: Anatole Denis <natolumin@unverle.fr>
Diffstat (limited to 'dhcpv4/dhcpv4_test.go')
| -rw-r--r-- | dhcpv4/dhcpv4_test.go | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/dhcpv4/dhcpv4_test.go b/dhcpv4/dhcpv4_test.go index 3198a66..aa1b2f6 100644 --- a/dhcpv4/dhcpv4_test.go +++ b/dhcpv4/dhcpv4_test.go @@ -4,6 +4,7 @@ import ( "bytes" "net" "strconv" + "strings" "testing" "github.com/insomniacslk/dhcp/iana" @@ -181,6 +182,17 @@ func TestNewToBytes(t *testing.T) { require.Equal(t, expected, got) } +func TestToBytesStringTooLong(t *testing.T) { + d, err := New() + if err != nil { + t.Fatal(err) + } + d.ServerHostName = strings.Repeat("a", 256) + d.BootFileName = strings.Repeat("a", 256) + + require.NotPanics(t, func() { _ = d.ToBytes() }) +} + func TestGetOption(t *testing.T) { d, err := New() if err != nil { |