Wireshark-dev: Re: [Wireshark-dev] question about option82
From: "Guo, Fei" <Fei.Guo@xxxxxxxxxxxxx>
Date: Thu, 4 Nov 2010 00:15:25 +0100
sorry, I only check the latest 1.2.12 version. thank you very much.
 
WIND RIVER | China Development Center
Tel: 86-10-8477-8665  |  Fax: 86-10-64790367
 


From: wireshark-dev-bounces@xxxxxxxxxxxxx [mailto:wireshark-dev-bounces@xxxxxxxxxxxxx] On Behalf Of Jaap Keuter
Sent: 2010年11月3日 18:30
To: Developer support list for Wireshark
Subject: Re: [Wireshark-dev] question about option82

On Wed, 3 Nov 2010 06:25:33 +0100, "Guo, Fei" <Fei.Guo@xxxxxxxxxxxxx> wrote:

Hi All,
  
  I capture a option82 packet with wireshark, but it is displayed with malformed. and I check the code, the following in function "bootp_dhcp_decode_agent_info".
--------------------------------------------------------------------------
 case 7: /* 7   RADIUS Attributes Sub-option           [RFC4014] */
  proto_tree_add_text(v_tree, tvb, optoff, subopt_len + 2,
     "RADIUS Attributes: %s",
     tvb_bytes_to_str(tvb, suboptoff, subopt_len));
  break;    --------------------------------   needed??
 case 8: /* 8   Authentication Suboption               [RFC4030]  */
  proto_tree_add_text(v_tree, tvb, optoff, subopt_len + 2,
     "Authentication: %s",
     tvb_bytes_to_str(tvb, suboptoff, subopt_len));
  break;    --------------------------------   needed??
 case 9:
  while (suboptoff < optend) {
   enterprise = tvb_get_ntohl(tvb, suboptoff);
   vti = proto_tree_add_text(v_tree, tvb, suboptoff, 4,
         "Enterprise-number: %s-%u",
         val_to_str( enterprise, sminmpec_values, "Unknown"),
         enterprise);
   suboptoff += 4;
--------------------------------------------------------------------------
  is it a bug?? if not, does RFC4030 show it need a "Enterprise-number" in the authentication sub-option?? thank you very much.
  
Cheers
Fei
 
WIND RIVER | China Development Center
Tel: 86-10-8477-8665  |  Fax: 86-10-64790367
 

Hi,

 

You're looking at old code; this was resolved over a year ago: http://anonsvn.wireshark.org/viewvc?view=rev&revision=30119

Thanks,
Jaap