Commit bf15ba7c authored by Martin Brandenburg's avatar Martin Brandenburg Committed by Mike Marshall

orangefs: skip forward to the next directory entry if seek is short

If userspace seeks to a position in the stream which is not correct, it
would have returned EIO because the data in the buffer at that offset
would be incorrect.  This and the userspace daemon returning a corrupt
directory are indistinguishable.

Now if the data does not look right, skip forward to the next chunk and
try again.  The motivation is that if the directory changes, an
application may seek to a position that was valid and no longer is valid.

It is not yet possible for a directory to change.
Signed-off-by: default avatarMartin Brandenburg <martin@omnibond.com>
Signed-off-by: default avatarMike Marshall <hubcap@omnibond.com>
parent 907bfcd8
...@@ -194,11 +194,18 @@ static int fill_from_part(struct orangefs_dir_part *part, ...@@ -194,11 +194,18 @@ static int fill_from_part(struct orangefs_dir_part *part,
/* The file offset from userspace is too large. */ /* The file offset from userspace is too large. */
if (i > part->len) if (i > part->len)
return -EIO; return 1;
/*
* If the seek pointer is positioned just before an entry it
* should find the next entry.
*/
if (i % 8)
i = i + (8 - i%8)%8;
while (i < part->len) { while (i < part->len) {
if (part->len < i + sizeof *len) if (part->len < i + sizeof *len)
return -EIO; break;
len = (void *)part + offset + i; len = (void *)part + offset + i;
/* /*
* len is the size of the string itself. padlen is the * len is the size of the string itself. padlen is the
...@@ -207,10 +214,10 @@ static int fill_from_part(struct orangefs_dir_part *part, ...@@ -207,10 +214,10 @@ static int fill_from_part(struct orangefs_dir_part *part,
padlen = (sizeof *len + *len + 1) + padlen = (sizeof *len + *len + 1) +
(8 - (sizeof *len + *len + 1)%8)%8; (8 - (sizeof *len + *len + 1)%8)%8;
if (part->len < i + padlen + sizeof *khandle) if (part->len < i + padlen + sizeof *khandle)
return -EIO; goto next;
s = (void *)part + offset + i + sizeof *len; s = (void *)part + offset + i + sizeof *len;
if (s[*len] != 0) if (s[*len] != 0)
return -EIO; goto next;
khandle = (void *)part + offset + i + padlen; khandle = (void *)part + offset + i + padlen;
if (!dir_emit(ctx, s, *len, if (!dir_emit(ctx, s, *len,
orangefs_khandle_to_ino(khandle), orangefs_khandle_to_ino(khandle),
...@@ -220,6 +227,9 @@ static int fill_from_part(struct orangefs_dir_part *part, ...@@ -220,6 +227,9 @@ static int fill_from_part(struct orangefs_dir_part *part,
i = i + (8 - i%8)%8; i = i + (8 - i%8)%8;
BUG_ON(i > part->len); BUG_ON(i > part->len);
ctx->pos = (ctx->pos & PART_MASK) | i; ctx->pos = (ctx->pos & PART_MASK) | i;
continue;
next:
i += 8;
} }
return 1; return 1;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment