diff options
author | Jan Kara <jack@suse.cz> | 2010-05-13 12:05:15 -0400 |
---|---|---|
committer | Jan Kara <jack@suse.cz> | 2010-05-21 13:30:48 -0400 |
commit | 741e128933448e589a85286e535078b24f4cf568 (patch) | |
tree | 40a069e5c7484673529f0747a9995cc7d2a85f00 /fs/ocfs2/quota.h | |
parent | 832d09cf1438bd172f69478bde74f20f05ec0115 (diff) |
ocfs2: Fix NULL pointer deref when writing local dquot
commit_dqblk() can write quota info to global file. That is actually a bad
thing to do because if we are just modifying local quota file, we are not
prepared (do not hold proper locks, do not have transaction credits) to do
a modification of the global quota file. So do not use commit_dqblk() and
instead call our writing function directly.
Acked-by: Joel Becker <Joel.Becker@oracle.com>
Signed-off-by: Jan Kara <jack@suse.cz>
Diffstat (limited to 'fs/ocfs2/quota.h')
-rw-r--r-- | fs/ocfs2/quota.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/fs/ocfs2/quota.h b/fs/ocfs2/quota.h index 903ffa933d53..196fcb52d95d 100644 --- a/fs/ocfs2/quota.h +++ b/fs/ocfs2/quota.h | |||
@@ -109,6 +109,7 @@ int ocfs2_read_quota_phys_block(struct inode *inode, u64 p_block, | |||
109 | struct buffer_head **bh); | 109 | struct buffer_head **bh); |
110 | int ocfs2_create_local_dquot(struct dquot *dquot); | 110 | int ocfs2_create_local_dquot(struct dquot *dquot); |
111 | int ocfs2_local_release_dquot(handle_t *handle, struct dquot *dquot); | 111 | int ocfs2_local_release_dquot(handle_t *handle, struct dquot *dquot); |
112 | int ocfs2_local_write_dquot(struct dquot *dquot); | ||
112 | 113 | ||
113 | extern const struct dquot_operations ocfs2_quota_operations; | 114 | extern const struct dquot_operations ocfs2_quota_operations; |
114 | extern struct quota_format_type ocfs2_quota_format; | 115 | extern struct quota_format_type ocfs2_quota_format; |