Call febootstrap-supermin-helper using the new -u and -g options
authorMatthew Booth <mbooth@redhat.com>
Thu, 28 Oct 2010 14:19:13 +0000 (15:19 +0100)
committerRichard W.M. Jones <rjones@redhat.com>
Thu, 28 Oct 2010 15:17:38 +0000 (16:17 +0100)
commit38af2eaceb4c8e9d675835bcd6e598ccb67daf0f
tree512159dc71c49b91b2c58ff0c71dfecc9eb19087
parentcf8add59d297d196facc2d0b9af45bb1d7946344
Call febootstrap-supermin-helper using the new -u and -g options

Use febootstrap-supermin-helper's new -u and -g command line options to setuid,
rather than doing it in libguestfs.

This resolves an issue with the generation of the cached appliance checksum. The
checksum was being generated by a call to febootstrap-supermin-helper through
popen(). Unfortunately, a bash misfeature meant that euid would be reset to uid,
and the checksum was generated for uid, not euid. When virt-v2v is writing to a
RHEV target, uid == 0 and euid == 36, which resulted in a cached appliance being
created for root with permissions for uid 36.

Note this requires febootstrap 2.10.
README
src/appliance.c