← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/item/empty_string.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/item/empty_string.h
17
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
18
18
 */
19
19
 
20
 
#ifndef DRIZZLED_ITEM_EMPTY_STRING_H
21
 
#define DRIZZLED_ITEM_EMPTY_STRING_H
 
20
#ifndef DRIZZLED_ITEM_EMPTRY_STRING_H
 
21
#define DRIZZLED_ITEM_EMPTRY_STRING_H
22
22
 
23
23
/**
24
24
  Item_empty_string -- is a utility class to put an item into List<Item>
28
28
 
29
29
#include <drizzled/item/string.h>
30
30
 
31
 
namespace drizzled
32
 
{
33
 
 
34
31
class Item_empty_string :public Item_string
35
32
{
36
33
public:
37
34
  Item_empty_string(const char *header,uint32_t length, const CHARSET_INFO * cs= NULL) :
38
35
    Item_string("",0, cs ? cs : &my_charset_utf8_general_ci)
39
 
    { name= const_cast<char*>(header); max_length= cs ? length * cs->mbmaxlen : length; }
 
36
    { name=(char*) header; max_length= cs ? length * cs->mbmaxlen : length; }
40
37
  void make_field(SendField *field);
41
38
};
42
 
 
43
 
} /* namespace drizzled */
44
39
  
45
 
#endif /* DRIZZLED_ITEM_EMPTY_STRING_H */
 
40
#endif /* DRIZZLED_ITEM_EMPTRY_STRING_H */