conf: Change virNWFilterObjSaveDef to virNWFilterSaveDef

There's no need to pass the driver pointer to nwfilter_conf, just
pass the configDir.
This commit is contained in:
John Ferlan 2017-02-28 17:12:09 -05:00
parent 15b881474b
commit 22426736b9
4 changed files with 9 additions and 11 deletions

View File

@ -3226,7 +3226,7 @@ virNWFilterLoadAllConfigs(virNWFilterObjListPtr nwfilters,
int int
virNWFilterObjSaveDef(virNWFilterDriverStatePtr driver, virNWFilterSaveDef(const char *configDir,
virNWFilterDefPtr def) virNWFilterDefPtr def)
{ {
char uuidstr[VIR_UUID_STRING_BUFLEN]; char uuidstr[VIR_UUID_STRING_BUFLEN];
@ -3234,17 +3234,15 @@ virNWFilterObjSaveDef(virNWFilterDriverStatePtr driver,
int ret = -1; int ret = -1;
char *configFile = NULL; char *configFile = NULL;
if (virFileMakePath(driver->configDir) < 0) { if (virFileMakePath(configDir) < 0) {
virReportSystemError(errno, virReportSystemError(errno,
_("cannot create config directory %s"), _("cannot create config directory %s"),
driver->configDir); configDir);
goto error; goto error;
} }
if (!(configFile = virFileBuildPath(driver->configDir, if (!(configFile = virFileBuildPath(configDir, def->name, ".xml")))
def->name, ".xml"))) {
goto error; goto error;
}
if (!(xml = virNWFilterDefFormat(def))) { if (!(xml = virNWFilterDefFormat(def))) {
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,

View File

@ -611,7 +611,7 @@ virNWFilterObjPtr virNWFilterObjFindByName(virNWFilterObjListPtr nwfilters,
const char *name); const char *name);
int virNWFilterObjSaveDef(virNWFilterDriverStatePtr driver, int virNWFilterSaveDef(const char *configDir,
virNWFilterDefPtr def); virNWFilterDefPtr def);
int virNWFilterObjDeleteDef(const char *configDir, int virNWFilterObjDeleteDef(const char *configDir,

View File

@ -753,7 +753,6 @@ virNWFilterObjFindByUUID;
virNWFilterObjListFree; virNWFilterObjListFree;
virNWFilterObjLock; virNWFilterObjLock;
virNWFilterObjRemove; virNWFilterObjRemove;
virNWFilterObjSaveDef;
virNWFilterObjUnlock; virNWFilterObjUnlock;
virNWFilterPrintStateMatchFlags; virNWFilterPrintStateMatchFlags;
virNWFilterPrintTCPFlags; virNWFilterPrintTCPFlags;
@ -765,6 +764,7 @@ virNWFilterRuleIsProtocolEthernet;
virNWFilterRuleIsProtocolIPv4; virNWFilterRuleIsProtocolIPv4;
virNWFilterRuleIsProtocolIPv6; virNWFilterRuleIsProtocolIPv6;
virNWFilterRuleProtocolTypeToString; virNWFilterRuleProtocolTypeToString;
virNWFilterSaveDef;
virNWFilterTestUnassignDef; virNWFilterTestUnassignDef;
virNWFilterUnlockFilterUpdates; virNWFilterUnlockFilterUpdates;
virNWFilterUnRegisterCallbackDriver; virNWFilterUnRegisterCallbackDriver;

View File

@ -555,7 +555,7 @@ nwfilterDefineXML(virConnectPtr conn,
if (!(nwfilter = virNWFilterObjAssignDef(&driver->nwfilters, def))) if (!(nwfilter = virNWFilterObjAssignDef(&driver->nwfilters, def)))
goto cleanup; goto cleanup;
if (virNWFilterObjSaveDef(driver, def) < 0) { if (virNWFilterSaveDef(driver->configDir, def) < 0) {
virNWFilterObjRemove(&driver->nwfilters, nwfilter); virNWFilterObjRemove(&driver->nwfilters, nwfilter);
def = NULL; def = NULL;
goto cleanup; goto cleanup;