[libvirt] [PATCH 0/3] storage: fix file unlink with old uid cache

Cole Robinson crobinso at redhat.com
Wed Mar 9 17:38:57 UTC 2016


There's been several reports of issues deleting storage volumes
after creating a VM with virt-manager/virt-install. These patches
fix the reports, see patch descriptions for details.

Patch #1 is the main fix IMO
Patch #2 is a related cleanup that clarifies some remove logic
Patch #3 is optional and may not be exhaustive, it's mostly for discussion


Pavel's original patches and discussions:

https://www.redhat.com/archives/libvir-list/2016-February/msg01224.html
https://bugzilla.redhat.com/show_bug.cgi?id=1289327
https://bugzilla.redhat.com/show_bug.cgi?id=1293804

Cole Robinson (3):
  storage: refresh volume before deletion
  util: virfile: Clarify setuid usage for virFileRemove
  util: virfile: Only setuid for virFileRemove if on NFS

 src/storage/storage_driver.c | 10 ++++++++++
 src/util/virfile.c           | 39 +++++++++++++++++++++++++++++++++------
 2 files changed, 43 insertions(+), 6 deletions(-)

-- 
2.5.0




More information about the libvir-list mailing list