From 1414db6ba6c7508bf6e0e4c1191218aa71d7ce43 Mon Sep 17 00:00:00 2001
From: "mkanat%kerio.com" <>
Date: Fri, 25 Feb 2005 11:28:01 +0000
Subject: Bug 282748: uninitialized value in localtime in Format.pm Patch By
 Frederic Buclin <LpSolit@gmail.com> r=wurblzap, a=myk

---
 Bugzilla/BugMail.pm |  8 +++-----
 Bugzilla/Util.pm    | 15 +++++++++------
 2 files changed, 12 insertions(+), 11 deletions(-)

(limited to 'Bugzilla')

diff --git a/Bugzilla/BugMail.pm b/Bugzilla/BugMail.pm
index df734ef6f..b4b0c9dfe 100644
--- a/Bugzilla/BugMail.pm
+++ b/Bugzilla/BugMail.pm
@@ -201,7 +201,9 @@ sub ProcessOneBug($) {
     }
     $values{'estimated_time'} = format_time_decimal($values{'estimated_time'});
 
-    $values{'deadline'} = time2str("%Y-%m-%d", str2time($values{'deadline'}));
+    if ($values{'deadline'}) {
+        $values{'deadline'} = time2str("%Y-%m-%d", str2time($values{'deadline'}));
+    }
 
     my @dependslist;
     SendSQL("SELECT dependson FROM dependencies WHERE 
@@ -262,10 +264,6 @@ sub ProcessOneBug($) {
                      WHERE attach_id = $attachid");
             $diffpart->{'isprivate'} = FetchOneColumn();
         }
-        if( $fieldname eq 'deadline' ) {
-            $old = time2str("%Y-%m-%d", str2time($old));
-            $new = time2str("%Y-%m-%d", str2time($new));
-        }  
         $difftext = FormatTriple($what, $old, $new);
         $diffpart->{'header'} = $diffheader;
         $diffpart->{'fieldname'} = $fieldname;
diff --git a/Bugzilla/Util.pm b/Bugzilla/Util.pm
index 3bc39ff09..7481c18c3 100644
--- a/Bugzilla/Util.pm
+++ b/Bugzilla/Util.pm
@@ -277,14 +277,17 @@ sub file_mod_time ($) {
 
 sub ValidateDate {
     my ($date, $format) = @_;
+    my $date2;
 
-    my $ts  = str2time($date);
-    my $date2 = time2str("%Y-%m-%d", $ts);
+    # $ts is undefined if the parser fails.
+    my $ts = str2time($date);
+    if ($ts) {
+        $date2 = time2str("%Y-%m-%d", $ts);
 
-    $date =~ s/(\d+)-0*(\d+?)-0*(\d+?)/$1-$2-$3/; 
-    $date2 =~ s/(\d+)-0*(\d+?)-0*(\d+?)/$1-$2-$3/;
-
-    if ($date ne $date2) {
+        $date =~ s/(\d+)-0*(\d+?)-0*(\d+?)/$1-$2-$3/; 
+        $date2 =~ s/(\d+)-0*(\d+?)-0*(\d+?)/$1-$2-$3/;
+    }
+    if (!$ts || $date ne $date2) {
         ThrowUserError('illegal_date', {date => $date, format => $format});
     } 
 }
-- 
cgit v1.2.3-24-g4f1b