diff options
author | Aruna Balakrishnaiah <aruna@linux.vnet.ibm.com> | 2013-06-25 05:03:56 -0400 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2013-06-25 13:04:38 -0400 |
commit | bf2883339a33b7544b92ea465b90c3de55082032 (patch) | |
tree | ade69be0d8eadfda272ef5c48eb38f448c8731c9 /fs/pstore | |
parent | 26446654584c096e23dbe5a9b3f641edf9190c9d (diff) |
pstore: Fail to unlink if a driver has not defined pstore_erase
pstore_erase is used to erase the record from the persistent store.
So if a driver has not defined pstore_erase callback return
-EPERM instead of unlinking a file as deleting the file without
erasing its record in persistent store will give a wrong impression
to customers.
Signed-off-by: Aruna Balakrishnaiah <aruna@linux.vnet.ibm.com>
Acked-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'fs/pstore')
-rw-r--r-- | fs/pstore/inode.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/pstore/inode.c b/fs/pstore/inode.c index e4bcb2cf055a..bfd95bf38005 100644 --- a/fs/pstore/inode.c +++ b/fs/pstore/inode.c | |||
@@ -178,6 +178,8 @@ static int pstore_unlink(struct inode *dir, struct dentry *dentry) | |||
178 | if (p->psi->erase) | 178 | if (p->psi->erase) |
179 | p->psi->erase(p->type, p->id, p->count, | 179 | p->psi->erase(p->type, p->id, p->count, |
180 | dentry->d_inode->i_ctime, p->psi); | 180 | dentry->d_inode->i_ctime, p->psi); |
181 | else | ||
182 | return -EPERM; | ||
181 | 183 | ||
182 | return simple_unlink(dir, dentry); | 184 | return simple_unlink(dir, dentry); |
183 | } | 185 | } |