Skip to content

Commit bc3ace6

Browse files
xzpeterFabiano Rosas
authored andcommitted
tests/migration: Drop arch_[source|target]
Coverity complained about them. These two variables are never used now after commit 832c732 ("migration-test: Create arch_opts"), and/or commit 34cc54f ("tests/qtest/migration-test: Use custom asm bios for ppc64"). Resolves: Coverity CID 1568379 Resolves: Coverity CID 1568380 Signed-off-by: Peter Xu <[email protected]> Reviewed-by: Alex Bennée <[email protected]> Message-Id: <[email protected]> Signed-off-by: Fabiano Rosas <[email protected]>
1 parent 9a9320a commit bc3ace6

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

tests/qtest/migration/framework.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -197,8 +197,6 @@ static void cleanup(const char *filename)
197197
int migrate_start(QTestState **from, QTestState **to, const char *uri,
198198
MigrateStart *args)
199199
{
200-
g_autofree gchar *arch_source = NULL;
201-
g_autofree gchar *arch_target = NULL;
202200
/* options for source and target */
203201
g_autofree gchar *arch_opts = NULL;
204202
g_autofree gchar *cmd_source = NULL;
@@ -307,12 +305,11 @@ int migrate_start(QTestState **from, QTestState **to, const char *uri,
307305
"-name source,debug-threads=on "
308306
"-m %s "
309307
"-serial file:%s/src_serial "
310-
"%s %s %s %s %s",
308+
"%s %s %s %s",
311309
kvm_opts ? kvm_opts : "",
312310
machine, machine_opts,
313311
memory_size, tmpfs,
314312
arch_opts ? arch_opts : "",
315-
arch_source ? arch_source : "",
316313
shmem_opts ? shmem_opts : "",
317314
args->opts_source ? args->opts_source : "",
318315
ignore_stderr);
@@ -329,12 +326,11 @@ int migrate_start(QTestState **from, QTestState **to, const char *uri,
329326
"-m %s "
330327
"-serial file:%s/dest_serial "
331328
"-incoming %s "
332-
"%s %s %s %s %s",
329+
"%s %s %s %s",
333330
kvm_opts ? kvm_opts : "",
334331
machine, machine_opts,
335332
memory_size, tmpfs, uri,
336333
arch_opts ? arch_opts : "",
337-
arch_target ? arch_target : "",
338334
shmem_opts ? shmem_opts : "",
339335
args->opts_target ? args->opts_target : "",
340336
ignore_stderr);

0 commit comments

Comments
 (0)