[libvirt] [PATCHv2] rpm: fix /var/lib/libvirt permissions

[Daniel Veillard]

On Tue, Nov 09, 2010 at 04:16:47PM -0700, Eric Blake wrote:
> https://bugzilla.redhat.com/show_bug.cgi?id=649511
> 
> Regression of forcing 0700 permissions (which breaks guest startup
> because the qemu user can't see /var/lib/libvirt/*.monitor) was
> introduced in commit 66823690e, as part of libvirt 0.8.2.
> 
> * libvirt.spec.in (%files): Drop %{_localstatedir}/lib/libvirt,
> since libvirt depends on libvirt-client.
> (%files client): Guarantee 755 permissions on
> %(_localstatedir}/lib/libvirt, since the qemu user must be able to
> do pathname resolution to a subdirectory.
> ---
> 
> v2 changes: don't declare directory twice
> (v1 at https://www.redhat.com/archives/libvir-list/2010-November/msg00238.html)
> 
>  libvirt.spec.in |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
> 
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index 813e0c0..f77626e 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -793,7 +793,6 @@ fi
> 
>  %dir %{_localstatedir}/run/libvirt/
> 
> -%dir %{_localstatedir}/lib/libvirt/
>  %dir %attr(0711, root, root) %{_localstatedir}/lib/libvirt/images/
>  %dir %attr(0711, root, root) %{_localstatedir}/lib/libvirt/boot/
>  %dir %attr(0700, root, root) %{_localstatedir}/cache/libvirt/
> @@ -883,7 +882,7 @@ fi
> 
>  %{_sysconfdir}/rc.d/init.d/libvirt-guests
>  %config(noreplace) %{_sysconfdir}/sysconfig/libvirt-guests
> -%dir %attr(0700, root, root) %{_localstatedir}/lib/libvirt
> +%dir %attr(0755, root, root) %{_localstatedir}/lib/libvirt/
> 
>  %if %{with_sasl}
>  %config(noreplace) %{_sysconfdir}/sasl2/libvirt.conf

ACK,

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel at veillard.com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/