aboutsummaryrefslogtreecommitdiff
path: root/changes/bug2337
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2010-12-07 14:37:50 -0500
committerNick Mathewson <nickm@torproject.org>2011-04-28 17:12:54 -0400
commit51e551d3837990b7c4491253a88bedd2513fe1de (patch)
tree7cf7c7166e57fa8882d8c38caca7226d1c0e1b49 /changes/bug2337
parent3055acbdbe914c31ca4825ed60b4cce6676bd61e (diff)
downloadtor-51e551d3837990b7c4491253a88bedd2513fe1de.tar
tor-51e551d3837990b7c4491253a88bedd2513fe1de.tar.gz
Detect and handle NULL returns from (gm/local)time_r
These functions can return NULL for otherwise-valid values of time_t. Notably, the glibc gmtime manpage says it can return NULL if the year if greater than INT_MAX, and the windows MSDN gmtime page says it can return NULL for negative time_t values. Also, our formatting code is not guaranteed to correctly handle years after 9999 CE. This patch tries to correct this by detecting NULL values from gmtime/localtime_r, and trying to clip them to a reasonable end of the scale. If they are in the middle of the scale, we call it a downright error. Arguably, it's a bug to get out-of-bounds dates like this to begin with. But we've had bugs of this kind in the past, and warning when we see a bug is much kinder than doing a NULL-pointer dereference. Boboper found this one too.
Diffstat (limited to 'changes/bug2337')
0 files changed, 0 insertions, 0 deletions