# HG changeset patch # User Daniel Atallah # Date 1219790243 0 # Node ID b789b54c5bb27805f87f164c5a9e8cc00a1d9795 # Parent 7071b52a0bf4986c37bc1fd91b9098b18fb06c98 purple_str_to_time() didn't necessarily set "rest" under all circumstances. I'm pretty sure this fixes #4525. diff -r 7071b52a0bf4 -r b789b54c5bb2 libpurple/log.c --- a/libpurple/log.c Tue Aug 26 22:08:21 2008 +0000 +++ b/libpurple/log.c Tue Aug 26 22:37:23 2008 +0000 @@ -887,9 +887,8 @@ struct tm tm; #if defined (HAVE_TM_GMTOFF) && defined (HAVE_STRUCT_TM_TM_ZONE) long tz_off; - const char *rest; + const char *rest, *end; time_t stamp = purple_str_to_time(purple_unescape_filename(filename), FALSE, &tm, &tz_off, &rest); - char *end; /* As zero is a valid offset, PURPLE_NO_TZ_OFF means no offset was * provided. See util.h. Yes, it's kinda ugly. */ diff -r 7071b52a0bf4 -r b789b54c5bb2 libpurple/util.c --- a/libpurple/util.c Tue Aug 26 22:08:21 2008 +0000 +++ b/libpurple/util.c Tue Aug 26 22:37:23 2008 +0000 @@ -768,6 +768,9 @@ time(&retval); localtime_r(&retval, &t); + if (rest != NULL) + *rest = NULL; + /* 4 digit year */ if (sscanf(c, "%04d", &year) && year > 1900) {