Subject: misc/19619: Typo in src/sys/dev/raidframe/rf_utils.c breaks raidframe kernel build
To: None <gnats-bugs@gnats.netbsd.org>
From: None <monroe@pobox.com>
List: netbsd-bugs
Date: 12/31/2002 15:53:11
>Number:         19619
>Category:       misc
>Synopsis:       Typo in src/sys/dev/raidframe/rf_utils.c breaks raidframe kernel build
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    misc-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Tue Dec 31 15:54:00 PST 2002
>Closed-Date:
>Last-Modified:
>Originator:     Monroe Williams
>Release:        current
>Organization:
>Environment:
N/A
>Description:
There's a typo in src/sys/dev/raidframe/rf_utils.c that prevents a kernel from building with the RF_INCLUDE_PARITY_DECLUSTERING option.  The problem is an extraneous "0)" in the source on line 96: 

-----
void
rf_free_1d_array(a, n)
    RF_RowCol_t *a;
    int     n;
{
    RF_Free(a, n * sizeof(RF_RowCol_t));
}
 0)
/* Euclid's algorithm:  finds and returns the greatest common divisor
 * between a and b.     (MCH)
 */
-----

It looks like this crept in between revisions 1.9 and 1.10, and was also pulled into 1.5.6.5 and 1.5.8.4.

>How-To-Repeat:
N/A
>Fix:
Remove the offending line.
>Release-Note:
>Audit-Trail:
>Unformatted: