← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/math/ord.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/math/ord.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/math/ord.h>
23
23
 
24
 
namespace drizzled
25
 
{
26
 
 
27
24
int64_t Item_func_ord::val_int()
28
25
{
29
26
  assert(fixed == 1);
35
32
  }
36
33
  null_value=0;
37
34
  if (!res->length()) return 0;
 
35
#ifdef USE_MB
38
36
  if (use_mb(res->charset()))
39
37
  {
40
38
    register const char *str=res->ptr();
45
43
      n=(n<<8)|(uint32_t)((unsigned char) *str++);
46
44
    return (int64_t) n;
47
45
  }
 
46
#endif
48
47
  return (int64_t) ((unsigned char) (*res)[0]);
49
48
}
50
49
 
51
 
} /* namespace drizzled */