Btrfs: fix readdir on 32 bit machines
authorChris Mason <chris.mason@oracle.com>
Wed, 28 Jan 2009 20:34:27 +0000 (15:34 -0500)
committerChris Mason <chris.mason@oracle.com>
Wed, 28 Jan 2009 20:34:27 +0000 (15:34 -0500)
After btrfs_readdir has gone through all the directory items, it
sets the directory f_pos to the largest possible int.  This way
applications that mix readdir with creating new files don't
end up in an endless loop finding the new directory items as they go.

It was a workaround for a bug in git, but the assumption was that if git
could make this looping mistake than it would be a common problem.

The largest possible int chosen was INT_LIMIT(typeof(file->f_pos),
and it is possible for that to be a larger number than 32 bit glibc
expects to come out of readdir.

This patches switches that to INT_LIMIT(off_t), which should keep
applications happy on 32 and 64 bit machines.

Signed-off-by: Chris Mason <chris.mason@oracle.com>
fs/btrfs/inode.c

index 288c2cdc7543ca7487c1831caf38b1191d9fbf4d..2bb65e9b1448e0898175eb75f4171e67d1e4ccbd 100644 (file)
@@ -3263,7 +3263,7 @@ skip:
 
        /* Reached end of directory/root. Bump pos past the last item. */
        if (key_type == BTRFS_DIR_INDEX_KEY)
-               filp->f_pos = INT_LIMIT(typeof(filp->f_pos));
+               filp->f_pos = INT_LIMIT(off_t);
        else
                filp->f_pos++;
 nopos: