kern/124723: Typo in error message in ggated.c
Garrett Cooper
yanefbsd at gmail.com
Thu Jun 19 03:50:13 UTC 2008
The following reply was made to PR kern/124723; it has been noted by GNATS.
From: "Garrett Cooper" <yanefbsd at gmail.com>
To: "Hywel Mallett" <Hywel at hmallett.co.uk>
Cc: bug-followup at freebsd.org
Subject: Re: kern/124723: Typo in error message in ggated.c
Date: Wed, 18 Jun 2008 20:41:15 -0700
On Wed, Jun 18, 2008 at 9:44 AM, Hywel Mallett <Hywel at hmallett.co.uk> wrote:
>
>>Number: 124723
>>Category: kern
>>Synopsis: Typo in error message in ggated.c
>>Confidential: no
>>Severity: non-critical
>>Priority: low
>>Responsible: freebsd-bugs
>>State: open
>>Quarter:
>>Keywords:
>>Date-Required:
>>Class: sw-bug
>>Submitter-Id: current-users
>>Arrival-Date: Wed Jun 18 16:50:01 UTC 2008
>>Closed-Date:
>>Last-Modified:
>>Originator: Hywel Mallett
>>Release: 7-RELENG
>>Organization:
>>Environment:
> FreeBSD one.option13.co.uk 7.0-STABLE FreeBSD 7.0-STABLE #0: Sun May 4 19:43:28 BST 2008 hywel at one.option13.co.uk:/usr/obj/usr/src/sys/GENERIC amd64
>
>>Description:
> In ggated there is an error message "No enough memory", which should (probably) read "Not enough memory".
>>How-To-Repeat:
>
>>Fix:
> Patch for src/sbin/ggate/ggated/ggated.c attached
>
>
> Patch attached with submission follows:
>
> --- /usr/src/sbin/ggate/ggated/ggated.c 2006-12-18 11:12:00.000000000 +0000
> +++ ggated.c 2008-06-18 17:39:26.000000000 +0100
> @@ -211,10 +211,10 @@
>
> ex = malloc(sizeof(*ex));
> if (ex == NULL)
> - g_gate_xlog("No enough memory.");
> + g_gate_xlog("Not enough memory.");
> ex->e_path = strdup(path);
> if (ex->e_path == NULL)
> - g_gate_xlog("No enough memory.");
> + g_gate_xlog("Not enough memory.");
>
> /* Made 'and' here. */
> ex->e_ip = (ip & mask);
That error message (IMO) is in and of itself misleading.
strerror(errno) with g_gate_xlog(...) would be a more suitable means
of denoting the error encountered.
-Garrett
More information about the freebsd-bugs
mailing list