Francois Gouget : testbot/LogUtils: Defend against invalid .errors lines.

Alexandre Julliard julliard at winehq.org
Tue Feb 25 16:02:34 CST 2020


Module: tools
Branch: master
Commit: 02a478745bb808c7c2cfbcbb5640d2dc0a1ec9f8
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=02a478745bb808c7c2cfbcbb5640d2dc0a1ec9f8

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Feb 25 00:14:09 2020 +0100

testbot/LogUtils: Defend against invalid .errors lines.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/lib/WineTestBot/LogUtils.pm | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index e309753..4cc8aaa 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -855,7 +855,13 @@ sub LoadLogErrors($)
     $LineNo++;
     chomp $Line;
     my ($Type, $Property, $Value) = split / /, $Line, 3;
-    if ($Type eq "p")
+    if (!defined $Value)
+    {
+      $LogInfo->{BadLog} = "$LineNo: Found an invalid line";
+      last;
+    }
+    # else $Type, $Property and $Value are all defined
+    elsif ($Type eq "p")
     {
       if (!defined $LogInfo->{$Property})
       {




More information about the wine-cvs mailing list