Date: Tue, 22 Jul 2003 14:56:37 -0500 From: "Jacques A. Vidrine" <nectar@FreeBSD.org> To: Simon Barner <barner@in.tum.de> Cc: Kris Kennaway <kris@obsecurity.org> Subject: Re: Fixing gcc 3.3 compile failures -- fix for net/netsaint-plugins Message-ID: <20030722195637.GC10922@madman.celabo.org> In-Reply-To: <20030722122608.GC552@zi025.glhnet.mhn.de> References: <20030718025200.GA36893@rot13.obsecurity.org> <20030722122608.GC552@zi025.glhnet.mhn.de>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jul 22, 2003 at 02:26:08PM +0200, Simon Barner wrote:
> --- plugins/check_by_ssh.c.orig Mon Apr 23 09:43:11 2001
> +++ plugins/check_by_ssh.c Tue Jul 22 05:05:52 2003
> @@ -191,7 +191,7 @@
> if (commands>1)
> remotecmd=strscat(remotecmd,";echo STATUS CODE: $?;");
>
> - if (strlen (remotecmd) <= 1)
> + if (remotecmd==NULL)
> usage ("No remotecmd\n");
>
> comm = ssprintf(comm,"%s %s '%s'",comm,hostname,remotecmd);
This looks like more than a fix for a `compile failure'. The
replacement code has different semantics than the new code.
What is the `compile failure' that was being fixed here?
Cheers,
--
Jacques Vidrine . NTT/Verio SME . FreeBSD UNIX . Heimdal
nectar@celabo.org . jvidrine@verio.net . nectar@freebsd.org . nectar@kth.se
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20030722195637.GC10922>