Как клонировать двумерный динамический массив строк?

Question

[Andrey]

Приветствую. Используя mysql возникла потребность в клонирование типа MYSQL_ROW row в другой динамический массив для длительного хранения.

mysql_t MysqlSelect(char *base, char *sql)
{
  if(base != NULL && sql != NULL)//входные данные
  {
    if(mysql.array != NULL)// Если было использование ранее то очистить
    {
      for(int step = 0; mysql.array[step]; step++)//очищаю каждый элемент массива
        free(mysql.array[step]);
      
      free(mysql.array);
      
      mysql.array_len = 0;// дефолтное количество элементов
    }
    
    for(int step = 0; step <= (int)strlen(sql); step++)// считаю сколько нужно данных для выборки
      if(sql[step] == ',')
        mysql.array_len++;
      
    mysql.connect = mysql_connection_setup(base);
    mysql.result = mysql_perform_query(mysql.connect, sql);
    
    if((mysql.row = mysql_fetch_row(mysql.result)) != NULL)
    {      
      mysql.array = (char**)malloc(sizeof(mysql.row) * sizeof(char**));// устанавливаю количество элементов
      
      for(int step = 0; step <= mysql.array_len; step++)
      {
        if(mysql.row[step] != NULL)
        {
          mysql.array[step] = (char*)calloc(sizeof(mysql.row[step]), sizeof(mysql.row[step]) * sizeof(char*)); //заполняю элементы
          strcpy(mysql.array[step], mysql.row[step]);
        }
      }
    }
    
    mysql_free_result(mysql.result);
    mysql_close(mysql.connect);
  }
  else
    printf("--> Log: input variables are nulled\n");
  
  return mysql;
}

В целом все работает, но valgrind указывает на утечки памяти, может есть более правильный метод реализации моей задачи?

Answer 1

[res2001]

for(int step = 0; mysql.array[step]; step++)//очищаю каждый элемент массива
        free(mysql.array[step]);

1.Цикл завершается на первом встреченном пустом элементе.
Судя по логике у вас могут быть пустые строки, под которые память не выделилась, а затем снова могут идти заполненные строки.
2.Кроме того после выделения памяти для mysql.array рекомендую сразу ее обнулять. malloc не зануляет выделенную память - в ней лежит мусор, используйте calloc или memset. Из-аз этого проверка в цикле выше (mysql.array[step]) может срабатывать не корректно.

Comments

[Andrey]

Подскажите пожалуйста, как правильно рассчитать память в данном случае для mysql.array и mysql.array[x]?

[res2001]

Allepta, по моему, память нужно выделять так:

mysql.array = (char**)calloc(mysql.array_len, sizeof(char**));

В первом цикле проходите весь массив и на каждом элементе вызывайте free() - она нормально проглатывает нулевые указатели.

[Andrey]

res2001, Опираясь на ваш первый ответ, не много изменил функцию и утечки памяти пока что пропали(более углубленно не тестировал), вот ее вид:

mysql_t MysqlSelect(char *base, char *sql)
{
  if(base != NULL && sql != NULL)
  {
    if(mysql.array != NULL)
    {     
      for(int step = 0; step <= mysql.array_len; step++)
        if(mysql.array[step] != NULL)
          free(mysql.array[step]);
        
      free(mysql.array);
      mysql.array_len = 0;
    }
    
    for(int step = 0; step < strlen(sql); step++)
      if(sql[step] == ',')
        mysql.array_len++;
            
    mysql.connect = mysql_connection_setup(base);
    mysql.result = mysql_perform_query(mysql.connect, sql);
    
    if((mysql.row = mysql_fetch_row(mysql.result)) != NULL)
    {
      long int size_str = 0;
      
      if(mysql.array_len == 0)
        size_str = strlen(mysql.row[0]) + 1;
      else
      {
        for(int step = 0; step <= mysql.array_len; step++)
          size_str = size_str + strlen(mysql.row[step]);
      }
            
      mysql.array = (char**)calloc(size_str, sizeof(char*));
      
      for(int step = 0; step <= mysql.array_len; step++)
      {
        mysql.array[step] = (char*)calloc(size_str, sizeof(char));
        strncpy(mysql.array[step], mysql.row[step], size_str);
      }
    }
    else
      mysql.array = NULL;
    
    mysql_free_result(mysql.result);
    mysql_close(mysql.connect);
  }
  else
    printf("--> Log: input variables are nulled\n");
  
  return mysql;
}

Как видите, я иначе решил выделить память, скажите правильно ли это?

[res2001]

Allepta, Зачем вы считаете этот size_str? В чем тут логика? Вы все равно выделяете память под каждую строку отдельно! При таком подходе size_str не нужен. Кроме того - это огромный перерасход памяти, особенно если много элементов в mysql.row.
Кроме того вы используете size_str для выделения памяти для mysql.array. ОПРЕДЕЛИТЕСЬ СКОЛЬКО ЭЛЕМЕНТОВ У ВАС БУДЕТ В mysql.array: size_str или mysql.array_len.
Повторюсь: вы выделяете память для size_str элементов в макссиве mysql.array, а используете из них только mysql.array_len.
Правильный вариант выделения памяти для mysql.array я писал выше.

Использовать size_str можно было бы, если бы вы выделяли память одним куском сразу для всех строк, а потом вычисляли бы указатель на каждую строку отдельно и складывали бы их в mysql.array. Это, кстати, не плохой вариант. Но нужно хорошо понимать и правильно выделять память и освобождать. Это может быть сложнее, чем текущий вариант, но работать будет быстрее.

[Andrey]

res2001, Понял. Благодарю за ответ, вы очень помогли.