diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2016-05-12 19:35:57 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2016-05-12 19:35:57 -0400 |
| commit | e82c3147554785414a10a2d424590758646972b2 (patch) | |
| tree | 54dcbe319a89601b153cfb8556faee1e6a6dda19 /fs/hpfs/hpfs_fn.h | |
| parent | 1d1bb236bc2ffb3586d6cc73e58c0d8351758123 (diff) | |
| download | lwn-e82c3147554785414a10a2d424590758646972b2.tar.gz lwn-e82c3147554785414a10a2d424590758646972b2.zip | |
hpfs: handle allocation failures in hpfs_add_pos()
pr_err() is nice, but we'd better propagate the error
to caller and not proceed to violate the invariants
(namely, "every file with f_pos tied to directory block
should have its address visible in per-inode array").
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/hpfs/hpfs_fn.h')
| -rw-r--r-- | fs/hpfs/hpfs_fn.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/hpfs/hpfs_fn.h b/fs/hpfs/hpfs_fn.h index 975654a63c13..aebb78f9e47f 100644 --- a/fs/hpfs/hpfs_fn.h +++ b/fs/hpfs/hpfs_fn.h @@ -242,7 +242,7 @@ extern const struct file_operations hpfs_dir_ops; /* dnode.c */ -void hpfs_add_pos(struct inode *, loff_t *); +int hpfs_add_pos(struct inode *, loff_t *); void hpfs_del_pos(struct inode *, loff_t *); struct hpfs_dirent *hpfs_add_de(struct super_block *, struct dnode *, const unsigned char *, unsigned, secno); |