]> xenbits.xensource.com Git - osstest.git/commitdiff
ts-livepatch-run: Fix erroneous $$ in double-check
authorIan Jackson <ian.jackson@eu.citrix.com>
Wed, 16 Jan 2019 11:32:06 +0000 (11:32 +0000)
committerIan Jackson <ian.jackson@eu.citrix.com>
Wed, 16 Jan 2019 11:36:11 +0000 (11:36 +0000)
The doubled $s here are simply a mistake.  The result is to make this
test ineffective, since `$$file' means `the value of the variable
whose name is in the variable $file', which here will never exist.
This produces a `Use of uninitialized value' warning and substitutes
the empty string, so overall we test the existence of the directory.

The missing check is not of much consequence since this check is not
really expected ever to fail, and if it does, some actual test
execution would fail due to the missing file.

So overall I think the only change is to log output.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>
CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
ts-livepatch-run

index f011e64e885d14f098b9e805fd058bb40a430315..86a79791b365e89fbc81602dcbdec65a6996f982 100755 (executable)
@@ -161,7 +161,7 @@ sub livepatch_test () {
 
 sub livepatch_check () {
     foreach my $file (@livepatch_files) {
-        if (!target_file_exists($ho, "/usr/lib/debug/xen-livepatch/$$file")) {
+        if (!target_file_exists($ho, "/usr/lib/debug/xen-livepatch/$file")) {
             die "$file is missing!\n";
         }
     }