diff options
author | Andreas Gruenbacher <agruenba@redhat.com> | 2018-07-06 23:05:41 +0100 |
---|---|---|
committer | Andreas Gruenbacher <agruenba@redhat.com> | 2018-07-25 00:06:48 +0200 |
commit | c25892827c7996eb19ca2a5b1cf596218122e994 (patch) | |
tree | 1210ec633eea5ea734e041e8c52e132c51c18801 /fs/gfs2 | |
parent | 4a7727725dc7d73769c5ab24c566df454093285f (diff) | |
download | op-kernel-dev-c25892827c7996eb19ca2a5b1cf596218122e994.zip op-kernel-dev-c25892827c7996eb19ca2a5b1cf596218122e994.tar.gz |
gfs2: fallocate_chunk: Always initialize struct iomap
In fallocate_chunk, always initialize the iomap before calling
gfs2_iomap_get_alloc: future changes could otherwise cause things like
iomap.flags to leak across calls.
Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
Reviewed-by: Bob Peterson <rpeterso@redhat.com>
Diffstat (limited to 'fs/gfs2')
-rw-r--r-- | fs/gfs2/file.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/gfs2/file.c b/fs/gfs2/file.c index f3c6d78..6d895d3 100644 --- a/fs/gfs2/file.c +++ b/fs/gfs2/file.c @@ -877,7 +877,6 @@ static int fallocate_chunk(struct inode *inode, loff_t offset, loff_t len, struct gfs2_inode *ip = GFS2_I(inode); loff_t end = offset + len; struct buffer_head *dibh; - struct iomap iomap = { }; int error; error = gfs2_meta_inode_buffer(ip, &dibh); @@ -893,6 +892,8 @@ static int fallocate_chunk(struct inode *inode, loff_t offset, loff_t len, } while (offset < end) { + struct iomap iomap = { }; + error = gfs2_iomap_get_alloc(inode, offset, end - offset, &iomap); if (error) |