Re: [libvirt] [PATCH] initgroups() in qemudOpenAsUID()
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.