On Wednesday 04 April 2001 04:03, Max Hammond wrote: > I'll just add that if anyone wants a dump of the data causing this, drop > me a note at [EMAIL PROTECTED] and I'll mail it to you, it's about 2500 > rows, and only that one field necessary to crash it. > > Cheers, > > Max Fix: --- 1.11/sql/time.cc Wed Nov 15 13:59:54 2000 +++ edited/time.cc Thu Apr 5 17:48:04 2001 @@ -428,11 +428,15 @@ uint field_length,year_length,digits,i,number_of_fields,date[7]; bool date_used=0; const char *pos; - const char *end=str+length; DBUG_ENTER("str_to_TIME"); DBUG_PRINT("enter",("str: %.*s",length,str)); - for (; !isdigit(*str) && str != end ; str++) ; // Skipp garbage + if(!str) + DBUG_RETURN(TIMESTAMP_NONE); + + const char *end=str+length; + + for (; !isdigit(*str) && str != end ; str++) ; // Skip garbage if (str == end) DBUG_RETURN(TIMESTAMP_NONE); /* The problem was actually not in deally with invalid wannabe-fuzzy dates, but rather in converting an empty string text/blob field to a date. Monty - I have considered several ways to fix it, and decided that checking for the NULL pointer in str_to_TIME() was the safest - it would take care of other potential buggy calls, if there happen to be any. -- MySQL Development Team __ ___ ___ ____ __ / |/ /_ __/ __/ __ \/ / Sasha Pachev <[EMAIL PROTECTED]> / /|_/ / // /\ \/ /_/ / /__ MySQL AB, http://www.mysql.com/ /_/ /_/\_, /___/\___\_\___/ Provo, Utah, USA <___/ --------------------------------------------------------------------- Before posting, please check: http://www.mysql.com/manual.php (the manual) http://lists.mysql.com/ (the list archive) To request this thread, e-mail <[EMAIL PROTECTED]> To unsubscribe, e-mail <[EMAIL PROTECTED]> Trouble unsubscribing? Try: http://lists.mysql.com/php/unsubscribe.php