diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2012-11-10 21:46:06 +0000 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2012-11-10 21:46:06 +0000 |
commit | 5c30c4715b1df29774a81fb21a942c3ae2588101 (patch) | |
tree | 471855e770a2eb3c719de302b81110cf8d461644 /epan/dissectors/packet-bootp.c | |
parent | a05901419528037616f18e26d1570d5fe2b96d02 (diff) |
From Jérôme LAFORGE via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7966 :
[DHCPv4] Fix bad decoding on option 119 with more than 3 fqdn
svn path=/trunk/; revision=45993
Diffstat (limited to 'epan/dissectors/packet-bootp.c')
-rw-r--r-- | epan/dissectors/packet-bootp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-bootp.c b/epan/dissectors/packet-bootp.c index 592c341a7d..9579bb84e9 100644 --- a/epan/dissectors/packet-bootp.c +++ b/epan/dissectors/packet-bootp.c @@ -2197,7 +2197,7 @@ bootp_option(tvbuff_t *tvb, packet_info *pinfo, proto_tree *bp_tree, int voff, while (offset < tvb_length(dns_domain_search_list.buff)) { /* use the get_dns_name method that manages all techniques of RFC 1035 (compression pointer and so on) */ - consumed = get_dns_name(dns_domain_search_list.buff, consumed, tvb_length(dns_domain_search_list.buff), 0, &dns_name); + consumed = get_dns_name(dns_domain_search_list.buff, offset, tvb_length(dns_domain_search_list.buff), 0, &dns_name); if (dns_domain_search_list.nb_option_119 == 1) { /* RFC 3396 is not used, so we can easily link the fqdn with v_tree. */ proto_tree_add_string(v_tree, hf_bootp_option_dhcp_dns_domain_search_list_fqdn, tvb, optoff + offset, consumed, dns_name); |