There are two "fix sfep->namelen" cases, but we only mark *dino_dirty = 1 in one of them. Add the other to ensure that the change gets written out. Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> --- diff --git a/repair/dir2.c b/repair/dir2.c index 6b8964d..25793e9 100644 --- a/repair/dir2.c +++ b/repair/dir2.c @@ -881,6 +881,7 @@ _("entry \"%*.*s\" in shortform directory %" PRIu64 " references %s inode %" PRI _("zero length entry in shortform dir %" PRIu64 ", resetting to %d\n"), ino, namelen); sfep->namelen = namelen; + *dino_dirty = 1; } else { do_warn( _("zero length entry in shortform dir %" PRIu64 ", would set to %d\n"), _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs