Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

check_dhcp: patch for "Rogue DHCP server detection" [sf#1854415] #752

Closed
monitoring-user opened this issue Sep 24, 2013 · 2 comments · Fixed by #1906
Closed

check_dhcp: patch for "Rogue DHCP server detection" [sf#1854415] #752

monitoring-user opened this issue Sep 24, 2013 · 2 comments · Fixed by #1906
Assignees
@riskersen
Labels
check_dhcp enhancement import patch

Comments

@monitoring-user
Copy link

Submitted by pcervicek on 2007-12-19 22:03:04

This patch helps check_dhcp to detect rogue dhcp servers.
Use it with the "-x" flag

example output:

[root pc4 plugins-root]# ./check_dhcp -s 192.168.1.1 -x
CRITICAL: Rogue DHCP Server detected! Server 192.168.1.205 offered 192.168.1.239
@monitoring-user
Copy link
Author

Added File: https://www.monitoring-plugins.org/attachments/259195-check_dhcp-roguedhcpservercheck.txt

@riskersen
Copy link
Collaborator

Correct link: http://sourceforge.net/p/nagiosplug/patches/272/

Rogue detection seems like a good idea..

@riskersen riskersen self-assigned this Oct 7, 2015
riskersen added a commit to riskersen/monitoring-plugins that referenced this issue Oct 7, 2015
@riskersen
* check_dhcp.c added patch from monitoring-plugins#752
7917c79
@riskersen riskersen mentioned this issue Oct 7, 2015
Closed
riskersen added a commit to riskersen/monitoring-plugins that referenced this issue Oct 7, 2015
@riskersen
* check_dhcp.c: reverted return code
963750a 
  - help and version should return UNKNOWN
  - option "?" not registered in getopt and will never match
	Invalid Option -- ?
  - closes monitoring-plugins#752
riskersen added a commit to riskersen/monitoring-plugins that referenced this issue Oct 7, 2015
@riskersen
* check_dhcp.c: merged patch monitoring-plugins#752
e0b1d93 
  - added dhcp rogue detection
  - closes monitoring-plugins#752
riskersen added a commit to riskersen/monitoring-plugins that referenced this issue Oct 9, 2015
@riskersen
check_dhcp.c merged patch from monitoring-plugins#752
ff400ad 
  - added dhcp rogue detection
	contributed by Patrick Cervicek (patrick AT cervicek.de)
  - closes monitoring-plugins#752
riskersen pushed a commit to riskersen/monitoring-plugins that referenced this issue Oct 9, 2015
@riskersen
check_dhcp.c merged patch from monitoring-plugins#752
915c5bd 
  - added dhcp rogue detection
	contributed by Patrick Cervicek (patrick AT cervicek.de)
  - closes monitoring-plugins#752
RincewindsHat pushed a commit that referenced this issue Aug 27, 2023
@RincewindsHat
check_dhcp.c merged patch from #752
4570de4 
  - added dhcp rogue detection
	contributed by Patrick Cervicek (patrick AT cervicek.de)
  - closes #752
@RincewindsHat RincewindsHat mentioned this issue Aug 27, 2023
Merged
RincewindsHat pushed a commit that referenced this issue Oct 1, 2023
@RincewindsHat
check_dhcp.c merged patch from #752
11670da 
  - added dhcp rogue detection
	contributed by Patrick Cervicek (patrick AT cervicek.de)
  - closes #752
RincewindsHat pushed a commit that referenced this issue Oct 3, 2023
@RincewindsHat
check_dhcp.c merged patch from #752
65237fd 
  - added dhcp rogue detection
	contributed by Patrick Cervicek (patrick AT cervicek.de)
  - closes #752
RincewindsHat added a commit that referenced this issue Oct 13, 2023
@RincewindsHat
Merge pull request #1906 from monitoring-plugins/check_dhcp_rogue_det…
60bdbe5 
…ection

check_dhcp: merged patch from #752
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@riskersen riskersen

Labels
check_dhcp enhancement import patch
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

check_dhcp: merged patch from #752 monitoring-plugins/monitoring-plugins
3 participants
@monitoring-user @riskersen @RincewindsHat