diff options
| author | Max Reitz <mreitz@redhat.com> | 2018-11-16 17:45:25 +0100 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2018-11-19 14:32:01 +0100 |
| commit | 577a133988c76e4ebf01d050d0d758d207a1baf7 (patch) | |
| tree | 73467460d0e846cfa5cf82224decb6a25f604638 /block/file-posix.c | |
| parent | 9ad08c44566bf4466c6263c71d43e9f7a354d4ba (diff) | |
| download | qemu-577a133988c76e4ebf01d050d0d758d207a1baf7.zip qemu-577a133988c76e4ebf01d050d0d758d207a1baf7.tar.gz qemu-577a133988c76e4ebf01d050d0d758d207a1baf7.tar.bz2 | |
file-posix: Fix shared locks on reopen commit
s->locked_shared_perm is the set of bits locked in the file, which is
the inverse of the permissions actually shared. So we need to pass them
as they are to raw_apply_lock_bytes() instead of inverting them again.
Reported-by: Alberto Garcia <berto@igalia.com>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Alberto Garcia <berto@igalia.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'block/file-posix.c')
| -rw-r--r-- | block/file-posix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/file-posix.c b/block/file-posix.c index 58c86a0..07bbdab 100644 --- a/block/file-posix.c +++ b/block/file-posix.c @@ -959,7 +959,7 @@ static void raw_reopen_commit(BDRVReopenState *state) /* Copy locks to the new fd before closing the old one. */ raw_apply_lock_bytes(NULL, rs->fd, s->locked_perm, - ~s->locked_shared_perm, false, &local_err); + s->locked_shared_perm, false, &local_err); if (local_err) { /* shouldn't fail in a sane host, but report it just in case. */ error_report_err(local_err); |