← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to tests/t/mysql_protocol.test

Replace MAX_(DATE|TIME).*_WIDTH defines in definitions.h with real (and correct) static const members to Temporal types.

This fixes the buffer overflow in https://bugs.launchpad.net/drizzle/+bug/373468

It also removes a handwritten snprintf in field/datetime.cc
However... this caused us to have to change Temporal to have a way to not
"convert" the int64_t value (so 20090101 becomes 20090101000000 etc) as it
has already been converted and we just want the Temporal type to do the
to_string conversion.

This still causes a failure in 'metadata' test due to size of timestamp type. I need feedback from Jay on when the usecond code comes into play to know the correct fix for this.

Show diffs side-by-side

added added

removed removed

Lines of Context:
tests/t/mysql_protocol.test
1
 
 
2
 
--disable_warnings
3
 
DROP TABLE IF EXISTS t1;
4
 
--enable_warnings
5
 
 
6
 
CREATE TABLE t1(a int, key (a));
7
 
INSERT INTO t1 VALUES (1), (2);
8
 
--exec $DRIZZLE_DUMP --skip-create --skip-comments test t1
9
 
DROP TABLE t1;
10
 
 
11
 
--exec $MYSQL --ping
12
 
 
13
 
--exec $DRIZZLE_SLAP --silent