← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/time/unix_timestamp.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/function/time/unix_timestamp.cc
17
17
 *  Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
18
18
 */
19
19
 
20
 
#include "config.h"
21
 
 
 
20
#include <drizzled/server_includes.h>
 
21
#include CSTDINT_H
22
22
#include <drizzled/function/time/unix_timestamp.h>
23
23
#include <drizzled/field/timestamp.h>
24
24
#include <drizzled/session.h>
25
25
 
26
26
#include "drizzled/temporal.h"
27
27
 
28
 
namespace drizzled
29
 
{
30
 
 
31
28
int64_t Item_func_unix_timestamp::val_int()
32
29
{
33
30
  DRIZZLE_TIME ltime;
53
50
    return 0;
54
51
  }
55
52
 
56
 
  Timestamp temporal;
 
53
  drizzled::Timestamp temporal;
57
54
 
58
55
  temporal.set_years(ltime.year);
59
56
  temporal.set_months(ltime.month);
66
63
  if (! temporal.is_valid())
67
64
  {
68
65
    null_value= true;
69
 
    char buff[DateTime::MAX_STRING_LENGTH];
 
66
    char buff[drizzled::DateTime::MAX_STRING_LENGTH];
70
67
    int buff_len;
71
 
    buff_len= temporal.to_string(buff, DateTime::MAX_STRING_LENGTH);
72
 
    assert((buff_len+1) < DateTime::MAX_STRING_LENGTH);
 
68
    buff_len= temporal.to_string(buff, drizzled::DateTime::MAX_STRING_LENGTH);
 
69
    assert((buff_len+1) < drizzled::DateTime::MAX_STRING_LENGTH);
73
70
    my_error(ER_INVALID_UNIX_TIMESTAMP_VALUE, MYF(0), buff);
74
71
    return 0;
75
72
  }
79
76
 
80
77
  return (int64_t) tmp;
81
78
}
82
 
 
83
 
} /* namespace drizzled */