diff options
| author |  Holger Weiss <holger@zedat.fu-berlin.de>
| 2012-06-13 12:36:42 +0200 |
|---|---|---|
| committer | Holger Weiss <holger@zedat.fu-berlin.de>
| 2012-06-13 12:36:42 +0200 |
| commit | f091d59e0f9abec9304d9d230fafc2dec001baba (patch) | |
| tree | 8485cbc75a5ec6cc90f97a21119d44797805607b /plugins-root | |
| parent | a80eafbf9cab9d80c526c504edf2ee4446a072d9 (diff) | |
| download | monitoring-plugins-f091d59e0f9abec9304d9d230fafc2dec001baba.tar.gz | |
check_dhcp: Fix handling of "pad" options
Don't let "pad" options[*] terminate the parsing of DHCP options. This
bug was triggered by using check_dhcp against Windows 2003 DHCP servers
(see #3503921).
[*] Cf. RFC 2132, 3.1.
Diffstat (limited to 'plugins-root')
| -rw-r--r-- | plugins-root/check_dhcp.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/plugins-root/check_dhcp.c b/plugins-root/check_dhcp.c index 2a1875c4..ac892747 100644 --- a/plugins-root/check_dhcp.c +++ b/plugins-root/check_dhcp.c @@ -839,8 +839,7 @@ int add_dhcp_offer(struct in_addr source,dhcp_packet *offer_packet){ /* process all DHCP options present in the packet */ for(x=4;x<MAX_DHCP_OPTIONS_LENGTH;){ - /* end of options (0 is really just a pad, but bail out anyway) */ - if((int)offer_packet->options[x]==-1 || (int)offer_packet->options[x]==0) + if((int)offer_packet->options[x]==-1) break; /* get option type */ @@ -872,7 +871,9 @@ int add_dhcp_offer(struct in_addr source,dhcp_packet *offer_packet){ } /* skip option data we're ignoring */ - if(option_type!=DHCP_OPTION_REBINDING_TIME) + if(option_type==0) /* "pad" option, see RFC 2132 (3.1) */ + x+=1; + else x+=option_length; } |