diff options
| author | Eric Wong <e@80x24.org> | 2020-08-07 10:42:52 +0000 |
|---|---|---|
| committer | Eric Wong <e@yhbt.net> | 2020-08-07 23:44:09 +0000 |
| commit | 607b26c426a89c843727a146ee66514efba3c99f (patch) | |
| tree | 04cee43a4932839a680c129a9ceeddbb3ce8db89 /lib/PublicInbox/View.pm | |
| parent | 2bfbb5943abef7fb2d5db08f51cf498aecff3030 (diff) | |
| download | public-inbox-607b26c426a89c843727a146ee66514efba3c99f.tar.gz | |
While we always generate YYYYMMDDhhmmss query parameters ourselves, the regexps in paginate_recent allow YYYYMMDD-only (no hhmmss) timestamps, so don't trigger Time::Local::timegm warnings about empty numeric comparisons on empty strings when a client starts making up their own URLs.
Diffstat (limited to 'lib/PublicInbox/View.pm')
| -rw-r--r-- | lib/PublicInbox/View.pm | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm index 4cb72bea..f23bfcc9 100644 --- a/lib/PublicInbox/View.pm +++ b/lib/PublicInbox/View.pm @@ -1147,7 +1147,7 @@ sub ts2str ($) { strftime('%Y%m%d%H%M%S', gmtime($_[0])) }; sub str2ts ($) { my ($yyyy, $mon, $dd, $hh, $mm, $ss) = unpack('A4A2A2A2A2A2', $_[0]); - timegm($ss, $mm, $hh, $dd, $mon - 1, $yyyy); + timegm($ss || 0, $mm || 0, $hh || 0, $dd, $mon - 1, $yyyy); } sub pagination_footer ($$) { |