[libvirt] [PATCH] Use virFileResolveLink instead of readlink in AppArmor
Jamie Strandboge
jamie at canonical.com
Thu Jan 21 17:24:06 UTC 2010
On Thu, 2010-01-21 at 11:33 -0500, Chris Lalancette wrote:
> --- a/src/security/security_apparmor.c
> +++ b/src/security/security_apparmor.c
> @@ -258,22 +258,23 @@ get_profile_name(virConnectPtr conn, virDomainObjPtr vm)
> static int
> use_apparmor(void)
> {
> - char libvirt_daemon[PATH_MAX];
> int rc = -1;
> - ssize_t len = 0;
> + char *libvirt_daemon = NULL;
>
> - if ((len = readlink("/proc/self/exe", libvirt_daemon,
> - PATH_MAX - 1)) < 0) {
> + if (virFileResolveLink("/proc/self/exe", &libvirt_daemon) < 0) {
> virSecurityReportError(NULL, VIR_ERR_INTERNAL_ERROR,
> "%s", _("could not find libvirtd"));
> - return rc;
> + return -1;
I've yet to test areadlink() from gnulib, but in the meantime, I'd
prefer if this were 'return rc' since rc is already '-1'.
Jamie
--
Jamie Strandboge | http://www.canonical.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100121/0ade1316/attachment-0001.sig>
More information about the libvir-list
mailing list