← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/statement/delete.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:
drizzled/statement/delete.cc
1
 
/* -*- mode: c++; c-basic-offset: 2; indent-tabs-mode: nil; -*-
2
 
 *  vim:expandtab:shiftwidth=2:tabstop=2:smarttab:
3
 
 *
4
 
 *  Copyright (C) 2009 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; either version 2 of the License, or
9
 
 *  (at your option) any later version.
10
 
 *
11
 
 *  This program is distributed in the hope that it will be useful,
12
 
 *  but WITHOUT ANY WARRANTY; without even the implied warranty of
13
 
 *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
14
 
 *  GNU General Public License for more details.
15
 
 *
16
 
 *  You should have received a copy of the GNU General Public License
17
 
 *  along with this program; if not, write to the Free Software
18
 
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
19
 
 */
20
 
 
21
 
#include "config.h"
22
 
#include <drizzled/show.h>
23
 
#include <drizzled/session.h>
24
 
#include <drizzled/lock.h>
25
 
#include <drizzled/probes.h>
26
 
#include <drizzled/statement/delete.h>
27
 
 
28
 
namespace drizzled
29
 
{
30
 
 
31
 
bool statement::Delete::execute()
32
 
{
33
 
  DRIZZLE_DELETE_START(session->query.c_str());
34
 
  TableList *first_table= (TableList *) session->lex->select_lex.table_list.first;
35
 
  TableList *all_tables= session->lex->query_tables;
36
 
  Select_Lex *select_lex= &session->lex->select_lex;
37
 
  Select_Lex_Unit *unit= &session->lex->unit;
38
 
  assert(first_table == all_tables && first_table != 0);
39
 
  assert(select_lex->offset_limit == 0);
40
 
  unit->set_limit(select_lex);
41
 
  bool need_start_waiting= false;
42
 
 
43
 
  if (! (need_start_waiting= ! wait_if_global_read_lock(session, 0, 1)))
44
 
  {
45
 
    return true;
46
 
  }
47
 
 
48
 
  bool res= mysql_delete(session, all_tables, select_lex->where,
49
 
                         &select_lex->order_list,
50
 
                         unit->select_limit_cnt, select_lex->options,
51
 
                         false);
52
 
  /*
53
 
    Release the protection against the global read lock and wake
54
 
    everyone, who might want to set a global read lock.
55
 
  */
56
 
  start_waiting_global_read_lock(session);
57
 
  return res;
58
 
}
59
 
 
60
 
} /* namespace drizzled */