-
Kevin Wolf authored
Commit 8ee03995 refactored the code incorrectly and broke the release of permissions on the old BDS. Instead of changing the permissions to the new required values after removing the old BDS from the list of children, it only re-obtains the permissions it already had. Change the order of operations so that the old BDS is removed again before calculating the new required permissions. Signed-off-by:
Kevin Wolf <kwolf@redhat.com>
Reviewed-by:
Eric Blake <eblake@redhat.com>
Reviewed-by:
Jeff Cody <jcody@redhat.com>
Reviewed-by:
John Snow <jsnow@redhat.com>Kevin Wolf authoredCommit 8ee03995 refactored the code incorrectly and broke the release of permissions on the old BDS. Instead of changing the permissions to the new required values after removing the old BDS from the list of children, it only re-obtains the permissions it already had. Change the order of operations so that the old BDS is removed again before calculating the new required permissions. Signed-off-by:
Kevin Wolf <kwolf@redhat.com>
Reviewed-by:
Eric Blake <eblake@redhat.com>
Reviewed-by:
Jeff Cody <jcody@redhat.com>
Reviewed-by:
John Snow <jsnow@redhat.com>
Loading