[Cluster-devel] cluster/gfs2/fsck metawalk.c

[rohara@sourceware.org <rohara@sourceware.org>]

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL5
Changes by:	rohara at sourceware.org	2006-11-20 15:44:53

Modified files:
	gfs2/fsck      : metawalk.c 

Log message:
	Fix another case where lf_dirent_format was not rewritten to disk
	after it was fixed.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/gfs2/fsck/metawalk.c.diff?cvsroot=cluster&only_with_tag=RHEL5&r1=1.3.2.2&r2=1.3.2.3

--- cluster/gfs2/fsck/metawalk.c	2006/11/16 19:56:09	1.3.2.2
+++ cluster/gfs2/fsck/metawalk.c	2006/11/20 15:44:53	1.3.2.3
@@ -162,8 +162,7 @@
 				}
 			}
 
-			enum update_flags u = not_updated;
-
+			*update = not_updated;
 			lbh = bread(sbp, leaf_no);
 			gfs2_leaf_in(&leaf, lbh->b_data);
 
@@ -177,7 +176,8 @@
 				log_debug("incorrect lf_dirent_format at leaf #%" PRIu64 "\n", leaf_no);
 				leaf.lf_dirent_format = GFS2_FORMAT_DE;
 				gfs2_leaf_out(&leaf, lbh->b_data);
-				u = updated;
+				log_debug("Fixing lf_dirent_format.\n");
+				*update = updated;
 			}
 
 			exp_count = (1 << (ip->i_di.di_depth - leaf.lf_depth));
@@ -227,9 +227,7 @@
 				 * compare it against leaf->lf_entries */
 				break;
 			} else {
-				if (u == updated)
-					log_debug("Fixing lf_dirent_format.\n");
-				brelse(lbh, u);
+				brelse(lbh, *updated);
 				if(!leaf.lf_next)
 					break;
 				leaf_no = leaf.lf_next;