← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to plugin/myisam/mi_static.cc

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:
plugin/myisam/mi_static.cc
1
 
/* Copyright (C) 2000-2002, 2004-2005 MySQL AB
2
 
 
3
 
   This program is free software; you can redistribute it and/or modify
4
 
   it under the terms of the GNU General Public License as published by
5
 
   the Free Software Foundation; version 2 of the License.
6
 
 
7
 
   This program is distributed in the hope that it will be useful,
8
 
   but WITHOUT ANY WARRANTY; without even the implied warranty of
9
 
   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
10
 
   GNU General Public License for more details.
11
 
 
12
 
   You should have received a copy of the GNU General Public License
13
 
   along with this program; if not, write to the Free Software
14
 
   Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA */
15
 
 
16
 
/*
17
 
  Static variables for MyISAM library. All definied here for easy making of
18
 
  a shared library
19
 
*/
20
 
 
21
 
#include "myisam_priv.h"
22
 
 
23
 
std::list<MI_INFO *> myisam_open_list;
24
 
unsigned char    myisam_file_magic[]=
25
 
{ (unsigned char) 254, (unsigned char) 254,'\007', '\001', };
26
 
unsigned char    myisam_pack_file_magic[]=
27
 
{ (unsigned char) 254, (unsigned char) 254,'\010', '\002', };
28
 
char * myisam_log_filename=(char*) "myisam.log";
29
 
int     myisam_log_file= -1;
30
 
uint    myisam_quick_table_bits=9;
31
 
uint32_t myisam_block_size= MI_KEY_BLOCK_LENGTH;                /* Best by test */
32
 
uint32_t myisam_concurrent_insert= 2;
33
 
uint32_t myisam_bulk_insert_tree_size=8192*1024;
34
 
uint32_t data_pointer_size=4;
35
 
 
36
 
/*
37
 
  read_vec[] is used for converting between P_READ_KEY.. and SEARCH_
38
 
  Position is , == , >= , <= , > , <
39
 
*/
40
 
 
41
 
uint32_t  myisam_read_vec[]=
42
 
{
43
 
  SEARCH_FIND, SEARCH_FIND | SEARCH_BIGGER, SEARCH_FIND | SEARCH_SMALLER,
44
 
  SEARCH_NO_FIND | SEARCH_BIGGER, SEARCH_NO_FIND | SEARCH_SMALLER,
45
 
  SEARCH_FIND | SEARCH_PREFIX, SEARCH_LAST, SEARCH_LAST | SEARCH_SMALLER,
46
 
  MBR_CONTAIN, MBR_INTERSECT, MBR_WITHIN, MBR_DISJOINT, MBR_EQUAL
47
 
};
48
 
 
49
 
uint32_t  myisam_readnext_vec[]=
50
 
{
51
 
  SEARCH_BIGGER, SEARCH_BIGGER, SEARCH_SMALLER, SEARCH_BIGGER, SEARCH_SMALLER,
52
 
  SEARCH_BIGGER, SEARCH_SMALLER, SEARCH_SMALLER
53
 
};