← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/errmsg_print.h

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:
drizzled/errmsg_print.h
25
25
#ifndef DRIZZLED_ERRMSG_PRINT_H
26
26
#define DRIZZLED_ERRMSG_PRINT_H
27
27
 
28
 
namespace drizzled
29
 
{
 
28
//void sql_print_error (const char *format, ...);
 
29
//void sql_print_warning (const char *format, ...);
 
30
//void sql_print_information (const char *format, ...);
 
31
//void sql_perror (const char *message);
30
32
 
31
33
#define ERRMSG_LVL_DBUG 1
32
34
#define ERRMSG_LVL_INFO 2
38
40
 
39
41
void sql_perror (const char *message);
40
42
 
41
 
} /* namespace drizzled */
42
 
 
43
43
#endif /* DRIZZLED_ERRMSG_PRINT_H */
44
44
 
45
45