← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/str/uuid.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/function/str/uuid.h
 
1
/* -*- mode: c++; c-basic-offset: 2; indent-tabs-mode: nil; -*-
 
2
 *  vim:expandtab:shiftwidth=2:tabstop=2:smarttab:
 
3
 *
 
4
 *  Copyright (C) 2008 Sun Microsystems
 
5
 *
 
6
 *  This program is free software; you can redistribute it and/or modify
 
7
 *  it under the terms of the GNU General Public License as published by
 
8
 *  the Free Software Foundation; version 2 of the License.
 
9
 *
 
10
 *  This program is distributed in the hope that it will be useful,
 
11
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
 
12
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 
13
 *  GNU General Public License for more details.
 
14
 *
 
15
 *  You should have received a copy of the GNU General Public License
 
16
 *  along with this program; if not, write to the Free Software
 
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
 
18
 */
 
19
 
 
20
#ifndef DRIZZLED_FUNCTION_STR_UUID_H
 
21
#define DRIZZLED_FUNCTION_STR_UUID_H
 
22
 
 
23
#include <drizzled/function/str/strfunc.h>
 
24
 
 
25
#define UUID_LENGTH (8+1+4+1+4+1+4+1+12)
 
26
class Item_func_uuid: public Item_str_func
 
27
{
 
28
public:
 
29
  Item_func_uuid(): Item_str_func() {}
 
30
  void fix_length_and_dec() {
 
31
    collation.set(system_charset_info);
 
32
    /*
 
33
       NOTE! uuid() should be changed to use 'ascii'
 
34
       charset when hex(), format(), md5(), etc, and implicit
 
35
       number-to-string conversion will use 'ascii'
 
36
    */
 
37
    max_length= UUID_LENGTH * system_charset_info->mbmaxlen;
 
38
  }
 
39
  const char *func_name() const{ return "uuid"; }
 
40
  String *val_str(String *);
 
41
};
 
42
 
 
43
#endif /* DRIZZLED_FUNCTION_STR_UUID_H */