← Back to branch summary

~drizzle-trunk/drizzle/development

« back to all changes in this revision

Viewing changes to drizzled/filesort.cc

  • Committer: Olaf van der Spek
  • Date: 2011-06-14 20:42:14 UTC
  • mto: (2318.6.3 refactor7)
  • mto: This revision was merged to the branch mainline in revision 2337.
  • Revision ID: olafvdspek@gmail.com-20110614204214-gswvc2b6fr113758
Remove malloc NULL check. Just die.

Show diffs side-by-side

added added

removed removed

Lines of Context:
drizzled/filesort.cc
248
248
  if (param.addon_field)
249
249
  {
250
250
    param.res_length= param.addon_length;
251
 
    if (!(table_sort.addon_buf= (unsigned char *) malloc(param.addon_length)))
252
 
    {
253
 
      goto err;
254
 
    }
 
251
    table_sort.addon_buf= (unsigned char *) malloc(param.addon_length);
255
252
  }
256
253
  else
257
254
  {
293
290
    selected_records_file= 0;
294
291
  }
295
292
 
296
 
  if (multi_byte_charset && !(param.tmp_buffer= (char*) malloc(param.sort_length)))
297
 
  {
298
 
    goto err;
299
 
  }
 
293
  if (multi_byte_charset)
 
294
    param.tmp_buffer= (char*) malloc(param.sort_length);
300
295
 
301
296
  memavl= getSession().variables.sortbuff_size;
302
297
  min_sort_memory= max((uint32_t)MIN_SORT_MEMORY, param.sort_length*MERGEBUFF2);
463
458
  char *char_pos;
464
459
 
465
460
  if (old_pos ||
466
 
      (old_pos= (char**) malloc((uint32_t) fields*(length+sizeof(char*)))))
 
461
      (old_pos= (char**) malloc((uint32_t) fields*(length+sizeof(char*))))) // todo: remove malloc check
467
462
  {
468
463
    pos=old_pos; char_pos=((char*) (pos+fields)) -length;
469
464
    while (fields--) *(pos++) = (char_pos+= length);
484
479
    return 0; /* sizeof(buffpek)*count will overflow */
485
480
  if (!tmp)
486
481
    tmp= (unsigned char *)malloc(length);
487
 
  if (tmp)
488
482
  {
489
483
    if (buffpek_pointers->reinit_io_cache(internal::READ_CACHE,0L,0,0) ||
490
484
        my_b_read(buffpek_pointers, (unsigned char*) tmp, length))
1072
1066
  if ((ha_rows) count > max_rows)
1073
1067
    count=(uint32_t) max_rows;
1074
1068
 
1075
 
  if (!(to= table_sort->record_pointers= (unsigned char*) malloc(res_length*count)))
1076
 
    return true;
 
1069
  to= table_sort->record_pointers= (unsigned char*) malloc(res_length*count);
1077
1070
 
1078
1071
  for (unsigned char **end_ptr= sort_keys+count ; sort_keys != end_ptr ; sort_keys++)
1079
1072
  {
1081
1074
    to+= res_length;
1082
1075
  }
1083
1076
 
1084
 
  return false;
 
1077
  return false; // return void
1085
1078
}
1086
1079
 
1087
1080
 
1612
1605
    return 0;
1613
1606
  length+= (null_fields+7)/8;
1614
1607
 
1615
 
  if (length+sortlength_arg > getSession().variables.max_length_for_sort_data ||
1616
 
      !(addonf= (sort_addon_field *) malloc(sizeof(sort_addon_field)*
1617
 
                                            (fields+1))))
 
1608
  if (length+sortlength_arg > getSession().variables.max_length_for_sort_data)
1618
1609
    return 0;
 
1610
  addonf= (sort_addon_field *) malloc(sizeof(sort_addon_field) * (fields+1));
1619
1611
 
1620
1612
  *plength= length;
1621
1613
  length= (null_fields+7)/8;