mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-03-07 17:28:15 +00:00
secdrivers: Rename @stdin_path argument of virSecurityDomainSetAllLabel()
The argument (if not NULL) points to the file the domain is restoring from. On QEMU command line this used to be '-incoming $path', but we've switched to passing FD ages ago and thus this argument is used only in AppArmor (which loads the profile on domain start). Anyway, the argument does not refer to stdin, rename it to 'incomingPath' then. Signed-off-by: Michal Privoznik <mprivozn@redhat.com> Reviewed-by: Erik Skultety <eskultet@redhat.com>
This commit is contained in:
parent
7e235954e5
commit
f68a14d17f
@ -455,7 +455,7 @@ AppArmorGenSecurityLabel(virSecurityManagerPtr mgr G_GNUC_UNUSED,
|
||||
static int
|
||||
AppArmorSetSecurityAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr def,
|
||||
const char *stdin_path,
|
||||
const char *incomingPath,
|
||||
bool chardevStdioLogd G_GNUC_UNUSED,
|
||||
bool migrated G_GNUC_UNUSED)
|
||||
{
|
||||
@ -464,10 +464,10 @@ AppArmorSetSecurityAllLabel(virSecurityManagerPtr mgr,
|
||||
if (!secdef || !secdef->relabel)
|
||||
return 0;
|
||||
|
||||
/* Reload the profile if stdin_path is specified. Note that
|
||||
/* Reload the profile if incomingPath is specified. Note that
|
||||
GenSecurityLabel() will have already been run. */
|
||||
if (stdin_path)
|
||||
return reload_profile(mgr, def, stdin_path, true);
|
||||
if (incomingPath)
|
||||
return reload_profile(mgr, def, incomingPath, true);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
@ -2142,7 +2142,7 @@ virSecurityDACSetSysinfoLabel(virSecurityManagerPtr mgr,
|
||||
static int
|
||||
virSecurityDACSetAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr def,
|
||||
const char *stdin_path G_GNUC_UNUSED,
|
||||
const char *incomingPath G_GNUC_UNUSED,
|
||||
bool chardevStdioLogd,
|
||||
bool migrated G_GNUC_UNUSED)
|
||||
{
|
||||
|
@ -82,7 +82,7 @@ typedef int (*virSecurityDomainReleaseLabel) (virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr sec);
|
||||
typedef int (*virSecurityDomainSetAllLabel) (virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr sec,
|
||||
const char *stdin_path,
|
||||
const char *incomingPath,
|
||||
bool chardevStdioLogd,
|
||||
bool migrated);
|
||||
typedef int (*virSecurityDomainRestoreAllLabel) (virSecurityManagerPtr mgr,
|
||||
|
@ -856,14 +856,14 @@ int virSecurityManagerCheckAllLabel(virSecurityManagerPtr mgr,
|
||||
int
|
||||
virSecurityManagerSetAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr vm,
|
||||
const char *stdin_path,
|
||||
const char *incomingPath,
|
||||
bool chardevStdioLogd,
|
||||
bool migrated)
|
||||
{
|
||||
if (mgr->drv->domainSetSecurityAllLabel) {
|
||||
int ret;
|
||||
virObjectLock(mgr);
|
||||
ret = mgr->drv->domainSetSecurityAllLabel(mgr, vm, stdin_path,
|
||||
ret = mgr->drv->domainSetSecurityAllLabel(mgr, vm, incomingPath,
|
||||
chardevStdioLogd,
|
||||
migrated);
|
||||
virObjectUnlock(mgr);
|
||||
|
@ -121,7 +121,7 @@ int virSecurityManagerCheckAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr sec);
|
||||
int virSecurityManagerSetAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr sec,
|
||||
const char *stdin_path,
|
||||
const char *incomingPath,
|
||||
bool chardevStdioLogd,
|
||||
bool migrated);
|
||||
int virSecurityManagerRestoreAllLabel(virSecurityManagerPtr mgr,
|
||||
|
@ -119,7 +119,7 @@ virSecurityDomainReleaseLabelNop(virSecurityManagerPtr mgr G_GNUC_UNUSED,
|
||||
static int
|
||||
virSecurityDomainSetAllLabelNop(virSecurityManagerPtr mgr G_GNUC_UNUSED,
|
||||
virDomainDefPtr sec G_GNUC_UNUSED,
|
||||
const char *stdin_path G_GNUC_UNUSED,
|
||||
const char *incomingPath G_GNUC_UNUSED,
|
||||
bool chardevStdioLogd G_GNUC_UNUSED,
|
||||
bool migrated G_GNUC_UNUSED)
|
||||
{
|
||||
|
@ -3169,7 +3169,7 @@ virSecuritySELinuxSetSysinfoLabel(virSecurityManagerPtr mgr,
|
||||
static int
|
||||
virSecuritySELinuxSetAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr def,
|
||||
const char *stdin_path G_GNUC_UNUSED,
|
||||
const char *incomingPath G_GNUC_UNUSED,
|
||||
bool chardevStdioLogd,
|
||||
bool migrated G_GNUC_UNUSED)
|
||||
{
|
||||
|
@ -341,7 +341,7 @@ virSecurityStackRestoreHostdevLabel(virSecurityManagerPtr mgr,
|
||||
static int
|
||||
virSecurityStackSetAllLabel(virSecurityManagerPtr mgr,
|
||||
virDomainDefPtr vm,
|
||||
const char *stdin_path,
|
||||
const char *incomingPath,
|
||||
bool chardevStdioLogd,
|
||||
bool migrated)
|
||||
{
|
||||
@ -350,7 +350,7 @@ virSecurityStackSetAllLabel(virSecurityManagerPtr mgr,
|
||||
|
||||
for (; item; item = item->next) {
|
||||
if (virSecurityManagerSetAllLabel(item->securityManager, vm,
|
||||
stdin_path, chardevStdioLogd,
|
||||
incomingPath, chardevStdioLogd,
|
||||
migrated) < 0)
|
||||
goto rollback;
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user