diff options
| author | Dan Carpenter <[email protected]> | 2020-08-24 11:58:12 +0300 |
|---|---|---|
| committer | David Howells <[email protected]> | 2020-10-27 22:05:56 +0000 |
| commit | 248c944e2159de4868bef558feea40214aaf8464 (patch) | |
| tree | ac60c074fc3ae494173904f93d9992fd070636fc | |
| parent | acc080d15dde820bd39eb55a04f9a09c7ef52e67 (diff) | |
afs: Fix a use after free in afs_xattr_get_acl()
The "op" pointer is freed earlier when we call afs_put_operation().
Fixes: e49c7b2f6de7 ("afs: Build an abstraction around an "operation" concept")
Signed-off-by: Dan Carpenter <[email protected]>
Signed-off-by: David Howells <[email protected]>
cc: Colin Ian King <[email protected]>
| -rw-r--r-- | fs/afs/xattr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/afs/xattr.c b/fs/afs/xattr.c index 84f3c4f57531..38884d6c57cd 100644 --- a/fs/afs/xattr.c +++ b/fs/afs/xattr.c @@ -85,7 +85,7 @@ static int afs_xattr_get_acl(const struct xattr_handler *handler, if (acl->size <= size) memcpy(buffer, acl->data, acl->size); else - op->error = -ERANGE; + ret = -ERANGE; } } |