kern/70815: bug in currently unused function

[Charlie &]

>Number:         70815
>Category:       kern
>Synopsis:       bug in currently unused function
>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:   Sun Aug 22 08:30:27 GMT 2004
>Closed-Date:
>Last-Modified:
>Originator:     info [at] haakh.de
>Release:        FreeBSD 5.2-CURRENT i386
>Organization:
Ingenieurbüro Haakh
>Environment:
System: FreeBSD abaton.Haakh.de 5.2-CURRENT FreeBSD 5.2-CURRENT #0: Wed Aug 18 15:53:28 CEST 2004 root at abaton.Haakh.de:/usr/src/sys/i386/compile/ABATON i386


>Description:
	Read-error not handeled due to checking the wrong variable
>How-To-Repeat:
	currently unused function
>Fix:
--- src/sys/boot/common/load.c.orig	Fri Dec 19 15:05:54 2003
+++ src/sys/boot/common/load.c	Sun Aug 22 10:15:24 2004
@@ -73,7 +73,7 @@
     /* Read everything until we know how big it is */
     for (;;) {
 	result = read(fd, buf, LOAD_TINYBUF);
-	if (size == -1) {
+	if (result == -1) {
 	    free(buf, LOAD_TINYBUF);
 	    close(fd);
 	    return(NULL);


>Release-Note:
>Audit-Trail:
>Unformatted: