[libvirt] [PATCH] initgroups() in qemudOpenAsUID()

Dan Kenigsberg danken at redhat.com
Tue Oct 19 10:11:05 UTC 2010


On Mon, Oct 18, 2010 at 01:21:05AM -0400, Laine Stump wrote:
>  On 10/17/2010 04:58 PM, Dan Kenigsberg wrote:
> >qemudOpenAsUID is intended to open a file with the credentials of a
> >specified uid. Current implementation fails if the file is accessible to
> >one of uid's groups but not owned by uid.
> >
> >This patch replaces the supplementary group list that the child process
> >inherited from libvirtd with the default group list of uid.
> 
> 
> Urr. Yet another twist in this ugly saga. Thanks for figuring it out!

The saga continues with another chapter. I've incorporated your
comments, and had to use the reentrant getpwuid_r after all. I also
bumped into another issue with saving to an already-existing file.





More information about the libvir-list mailing list