d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Removed the ``#if 0'' that turned bounds_check_with_label() into worse

Browse Source 
than a NOP.  bounds_check_with_label() would return -1 yet NOT set any
of the bio flags to show an error.  This meant the caller would not
properly see that bounds_check_with_label() did not do any work.  This
prevented newfs(8) from being able to write a file system on any partition
other than `c' on a `ccd'.

The logs of this file do not tell _why_ bounds_check_with_label() was
emasculated.  Nor are there any `XXX' comments.  So we'll unemasculated
it, and see what breaks.

Submitted by:	gallatin
This commit is contained in:
obrien 2001-03-29 20:18:45 +00:00
parent d11eb55932
commit a7dfa00bb3
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/alpha/alpha/machdep.c
View File

@ -2020,7 +2020,6 @@ Debugger(const char *msg)
int
bounds_check_with_label(struct bio *bp, struct disklabel *lp, int wlabel)
{
#if 0
struct partition *p = lp->d_partitions + dkpart(bp->bio_dev);
int labelsect = lp->d_partitions[0].p_offset;
int maxsz = p->p_size,
@ -2067,7 +2066,6 @@ bounds_check_with_label(struct bio *bp, struct disklabel *lp, int wlabel)
bad:
bp->bio_flags |= BIO_ERROR;
#endif
return(-1);
}