Merge pull request #4147 from datacompboy/patch-1

Fix ValidateDateTime: check day instead month

Merge pull request #4147 from datacompboy/patch-1
Fix ValidateDateTime: check day instead month
d4afdba8 · Adam Cozzette · GitHub · bab843b8 · 473c5cff · d4afdba8
Unverified Commit d4afdba8 authored Jan 08, 2018 by Adam Cozzette Committed by GitHub Jan 08, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

time.cc src/google/protobuf/stubs/time.cc +2 -2

No files found.
--- a/src/google/protobuf/stubs/time.cc
+++ b/src/google/protobuf/stubs/time.cc
@@ -80,9 +80,9 @@ bool ValidateDateTime(const DateTime& time) {
    return false;
  }
  if (time.month == 2 && IsLeapYear(time.year)) {
-    return time.month <= kDaysInMonth[time.month] + 1;
+    return time.day <= kDaysInMonth[time.month] + 1;
  } else {
-    return time.month <= kDaysInMonth[time.month];
+    return time.day <= kDaysInMonth[time.month];
  }
 }