← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/function/time/extract.cc

  • Committer: lbieber
  • Date: 2010-09-10 16:06:15 UTC
  • mfrom: (1754.1.1 build)
  • Revision ID: lbieber@orisndriz03-20100910160615-i9y0hqsnnl7atdky
Merge Shrews - fix bug 562349 - date and time functions corrupt blob data in pbxt engine

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/function/time/extract.cc
113
113
          char buff[DRIZZLE_MAX_LENGTH_DATETIME_AS_STRING];
114
114
          String tmp(buff,sizeof(buff), &my_charset_utf8_bin);
115
115
          String *res= args[0]->val_str(&tmp);
116
 
          if (! datetime_temporal.from_string(res->c_ptr(), res->length()))
 
116
 
 
117
          if (res && (res != &tmp))
 
118
          {
 
119
            tmp.copy(*res);
 
120
          }
 
121
 
 
122
          if (! datetime_temporal.from_string(tmp.c_ptr(), tmp.length()))
117
123
          {
118
124
            /* 
119
125
            * Could not interpret the function argument as a temporal value, 
120
126
            * so throw an error and return 0
121
127
            */
122
 
            my_error(ER_INVALID_DATETIME_VALUE, MYF(0), res->c_ptr());
 
128
            my_error(ER_INVALID_DATETIME_VALUE, MYF(0), tmp.c_ptr());
123
129
            return 0;
124
130
          }
125
131
        }
141
147
 
142
148
          res= args[0]->val_str(&tmp);
143
149
 
144
 
          my_error(ER_INVALID_DATETIME_VALUE, MYF(0), res->c_ptr());
 
150
          if (res && (res != &tmp))
 
151
          {
 
152
            tmp.copy(*res);
 
153
          }
 
154
 
 
155
          my_error(ER_INVALID_DATETIME_VALUE, MYF(0), tmp.c_ptr());
145
156
          return 0;
146
157
        }
147
158
    }
172
183
    char time_buff[DRIZZLE_MAX_LENGTH_DATETIME_AS_STRING];
173
184
    String tmp_time(time_buff,sizeof(time_buff), &my_charset_utf8_bin);
174
185
    String *time_res= args[0]->val_str(&tmp_time);
175
 
    if (! time_temporal.from_string(time_res->c_ptr(), time_res->length()))
 
186
 
 
187
    if (time_res && (time_res != &tmp_time))
 
188
    {
 
189
      tmp_time.copy(*time_res);
 
190
    }
 
191
 
 
192
    if (! time_temporal.from_string(tmp_time.c_ptr(), tmp_time.length()))
176
193
    {
177
194
      /* 
178
195
       * OK, we failed to match the first argument as a string
197
214
            char buff[DRIZZLE_MAX_LENGTH_DATETIME_AS_STRING];
198
215
            String tmp(buff,sizeof(buff), &my_charset_utf8_bin);
199
216
            String *res= args[0]->val_str(&tmp);
200
 
            if (! datetime_temporal.from_string(res->c_ptr(), res->length()))
 
217
 
 
218
            if (res && (res != &tmp))
 
219
            {
 
220
              tmp.copy(*res);
 
221
            }
 
222
 
 
223
            if (! datetime_temporal.from_string(tmp.c_ptr(), tmp.length()))
201
224
            {
202
225
              /* 
203
226
               * Could not interpret the function argument as a temporal value, 
204
227
               * so throw an error and return 0
205
228
               */
206
 
              my_error(ER_INVALID_DATETIME_VALUE, MYF(0), res->c_ptr());
 
229
              my_error(ER_INVALID_DATETIME_VALUE, MYF(0), tmp.c_ptr());
207
230
              return 0;
208
231
            }
209
232
          }