Fix some misleading comments in XM driver

This commit is contained in:
Daniel P. Berrange 2009-01-20 18:20:09 +00:00
parent 2c7be88d15
commit 84ef468ba8
2 changed files with 12 additions and 7 deletions

View File

@ -1,3 +1,8 @@
Tue Jan 20 18:16:53 GMT 2009 Daniel P. Berrange <berrange@redhat.com>
* src/xm_internal.c: Fix 2 misleading comments & potential
NULL deference
Tue Jan 20 17:46:53 GMT 2009 Daniel P. Berrange <berrange@redhat.com>
* src/remote_internal.c: Remove a use of strerror() in error

View File

@ -559,10 +559,10 @@ int xenXMConfigCacheRefresh (virConnectPtr conn) {
/*
* Open a 'connection' to the config file directory ;-)
* We just create a hash table to store config files in.
* We only support a single directory, so repeated calls
* to open all end up using the same cache of files
* The XM driver keeps a cache of config files as virDomainDefPtr
* objects in the xenUnifiedPrivatePtr. Optionally inotify lets
* us watch for changes (see separate driver), otherwise we poll
* every few seconds
*/
virDrvOpenStatus
xenXMOpen (virConnectPtr conn,
@ -591,8 +591,8 @@ xenXMOpen (virConnectPtr conn,
}
/*
* Free the config files in the cache if this is the
* last connection
* Free the cached config files associated with this
* connection
*/
int xenXMClose(virConnectPtr conn) {
xenUnifiedPrivatePtr priv = conn->privateData;
@ -1290,7 +1290,7 @@ no_memory:
* domain, suitable for later feeding for virDomainCreateXML
*/
char *xenXMDomainDumpXML(virDomainPtr domain, int flags) {
xenUnifiedPrivatePtr priv = domain->conn->privateData;
xenUnifiedPrivatePtr priv;
const char *filename;
xenXMConfCachePtr entry;