comparison m_option.c @ 9177:01a713dcaf23

disable free() in string and string_list parsers. yes, it's a hack (and a little memleak), but i can explain :) [note it's just a few kB memleak, but it's the price of stability without full code review/audit - there are hunderds of possible double free()] the old config parser didn't free() strings/stringlists, but didn't even allocate them by default. the new one always free(), and it causes memcorruption/sig11 at cases like this: char* dvd_device="/dev/dvd"; {"dvd-device", &dvd_device, CONF_TYPE_STRING, 0, 0, 0, NULL}, since string constansts (allocated in .TEXT segment) cannot be free()'d

9176:237b7e51521a

  return (val && VAL(val) && strlen(VAL(val)) > 0) ? strdup(VAL(val)) : NULL;
}

static void copy_str(m_option_t* opt,void* dst, void* src) {
  if(dst && src) {
    if(VAL(dst)) free(VAL(dst));
    VAL(dst) = VAL(src) ? strdup(VAL(src)) : NULL;
  }
}
  
static void free_str(void* src) {
  if(src && VAL(src)){ 
    free(VAL(src));
    VAL(src) = NULL;
  }
}

m_option_type_t m_option_type_string = {

  int i;

  if(!dst || !VAL(dst)) return;
  d = VAL(dst);


  for(i = 0 ; d[i] != NULL ; i++)
    free(d[i]);
  free(d);
  VAL(dst) = NULL;
}

static int str_list_add(char** add, int n,void* dst,int pre) {
  char** lst = VAL(dst);

9177:01a713dcaf23

  return (val && VAL(val) && strlen(VAL(val)) > 0) ? strdup(VAL(val)) : NULL;
}

static void copy_str(m_option_t* opt,void* dst, void* src) {
  if(dst && src) {
//    if(VAL(dst)) free(VAL(dst)); //FIXME!!!
    VAL(dst) = VAL(src) ? strdup(VAL(src)) : NULL;
  }
}
  
static void free_str(void* src) {
  if(src && VAL(src)){ 
//    free(VAL(src)); //FIXME!!!
    VAL(src) = NULL;
  }
}

m_option_type_t m_option_type_string = {

  int i;

  if(!dst || !VAL(dst)) return;
  d = VAL(dst);

// FIXME!!!
//  for(i = 0 ; d[i] != NULL ; i++)
//    free(d[i]);
//  free(d);
  VAL(dst) = NULL;
}

static int str_list_add(char** add, int n,void* dst,int pre) {
  char** lst = VAL(dst);