Update incorrect comments here, there should not be a call to panic()
over fs corruption. Discussed with: alfred, phk
This commit is contained in:
parent
cb5c1ec725
commit
9fc31f8a5f
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=154730
@ -1176,7 +1176,7 @@ msdosfs_rename(ap)
|
||||
zp->de_fndoffset = from_diroffset;
|
||||
error = removede(zp, ip);
|
||||
if (error) {
|
||||
/* XXX should really panic here, fs is corrupt */
|
||||
/* XXX should downgrade to ro here, fs is corrupt */
|
||||
if (newparent)
|
||||
VOP_UNLOCK(fdvp, 0, td);
|
||||
VOP_UNLOCK(fvp, 0, td);
|
||||
@ -1186,7 +1186,7 @@ msdosfs_rename(ap)
|
||||
error = pcbmap(dp, de_cluster(pmp, to_diroffset), 0,
|
||||
&ip->de_dirclust, 0);
|
||||
if (error) {
|
||||
/* XXX should really panic here, fs is corrupt */
|
||||
/* XXX should downgrade to ro here, fs is corrupt */
|
||||
if (newparent)
|
||||
VOP_UNLOCK(fdvp, 0, td);
|
||||
VOP_UNLOCK(fvp, 0, td);
|
||||
@ -1216,7 +1216,7 @@ msdosfs_rename(ap)
|
||||
error = bread(pmp->pm_devvp, bn, pmp->pm_bpcluster,
|
||||
NOCRED, &bp);
|
||||
if (error) {
|
||||
/* XXX should really panic here, fs is corrupt */
|
||||
/* XXX should downgrade to ro here, fs is corrupt */
|
||||
brelse(bp);
|
||||
VOP_UNLOCK(fvp, 0, td);
|
||||
goto bad;
|
||||
@ -1227,7 +1227,7 @@ msdosfs_rename(ap)
|
||||
putushort(dotdotp->deHighClust, dp->de_StartCluster >> 16);
|
||||
error = bwrite(bp);
|
||||
if (error) {
|
||||
/* XXX should really panic here, fs is corrupt */
|
||||
/* XXX should downgrade to ro here, fs is corrupt */
|
||||
VOP_UNLOCK(fvp, 0, td);
|
||||
goto bad;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user