Call $equivflagscheckq->finish() at the end, rather than in the candidate search loop. This avoids missing a `next' control path.
Call $resprop_q->finish() at all. One of these is responsible for this message: DBI::db=HASH(0x88e79c4)->disconnect invalidates 1 active statement handle (either destroy statement handles or call finish on them before disconnecting) at Osstest/Executive.pm line 708, <GEN4> line 53. Signed-off-by: Ian Jackson <[email protected]> --- ts-hosts-allocate-Executive | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ts-hosts-allocate-Executive b/ts-hosts-allocate-Executive index 88da504..13a0652 100755 --- a/ts-hosts-allocate-Executive +++ b/ts-hosts-allocate-Executive @@ -366,7 +366,6 @@ END push @{ $candrow->{Warnings} }, "host has multiple equivalence class flags"; } - $equivflagscheckq->finish(); } print DEBUG "$dbg FLAGS MISSINGFLAGS: @missingflags.\n"; @@ -419,7 +418,9 @@ END print DEBUG "$dbg CANDIDATE.\n"; } $findhostsq->finish(); + $equivflagscheckq->finish(); $equivstatusq->finish(); + $resprop_q->finish(); if (!@candidates) { if (defined $use) { -- 1.7.10.4 _______________________________________________ Xen-devel mailing list [email protected] http://lists.xen.org/xen-devel
