[Nagiosplug-devel] check_mysql cmd args fixed

[Dave Viner]

Hi all,
	Now that some of my cohorts have gotten this working, they are developing a
check test to verify the status of mysql replication.  Should I submit the
code to this list or would it be easier for me to become a committer for
this project?

	Also, has anyone had a chance to review my earlier patch to check_mysql?
Not a big deal, just a minor annoyance that is fixed.

thanks

dave


-----Original Message-----
From: nagiosplug-devel-admin at lists.sourceforge.net
[mailto:nagiosplug-devel-admin at lists.sourceforge.net]On Behalf Of Dave
Viner
Sent: Monday, December 09, 2002 3:02 PM
To: Nagiosplug-devel at lists.sourceforge.net
Subject: [Nagiosplug-devel] check_mysql cmd args fixed


Hi,
	I just downloaded and compiled the awesome plugins for Nagios from
sourceforge.  However, whenever I run check_mysql I get a coredump.  After
closer examinination, I discovered that there is a subtle error in the
processing of arguments from the command line.  I've attached a small patch
file which fixes this problem.  Essentially, this patch allows a user to
specify the host, user, password, database, and port in the arguments (but
not in the options).  I believe this is the intent of the code that is there
now.  This patch means that these two commands are identical:

	./check_mysql -H myhost.example.com -u myuser -p mypass -d mytbl -P 3306
	./check_mysql myhost.example.com myuser mypass mytbl 3306

dave.