External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-03-07 17:28:15 +00:00
Code

Remove bogus assert() from migration code

Browse Source 
assert() is forbidden in libvirt code, and these two cases would
in fact never execute due to earlier error checks.

* src/libvirt.c: Remove assert() usage
This commit is contained in:
Daniel P. Berrange 2011-04-20 11:39:51 +01:00
parent 604708df27
commit 8b83cc4219
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/libvirt.c
View File

@ -17,7 +17,6 @@
#include <sys/types.h> #include <sys/types.h>
#include <sys/stat.h> #include <sys/stat.h>
#include <unistd.h> #include <unistd.h>
#include <assert.h>
#include <sys/wait.h> #include <sys/wait.h>
#include <time.h> #include <time.h>
#include <gcrypt.h> #include <gcrypt.h>
@ -3374,7 +3373,6 @@ virDomainMigrateVersion1 (virDomainPtr domain,
} }
if (uri_out) if (uri_out)
uri = uri_out; /* Did domainMigratePrepare change URI? */ uri = uri_out; /* Did domainMigratePrepare change URI? */
assert (uri != NULL);
/* Perform the migration. The driver isn't supposed to return /* Perform the migration. The driver isn't supposed to return
* until the migration is complete. * until the migration is complete.
@ -3464,7 +3462,6 @@ virDomainMigrateVersion2 (virDomainPtr domain,
} }
if (uri_out) if (uri_out)
uri = uri_out; /* Did domainMigratePrepare2 change URI? */ uri = uri_out; /* Did domainMigratePrepare2 change URI? */
assert (uri != NULL);
/* Perform the migration. The driver isn't supposed to return /* Perform the migration. The driver isn't supposed to return
* until the migration is complete. * until the migration is complete.