освобождаемый указатель не был выделен (только OSX)

Приведенная ниже функция в основном эмулирует mkdir -p, рекурсивно создавая каталоги для заданного пути. С Linux у меня нет проблем, однако, работая под OSX, он всегда выдает ошибку pointer being freed was not allocated во время вызова free(dir). Кто-нибудь может заметить ошибку? Когда я выполняю выполнение в gdb, я не вижу никаких очевидных проблем, dir заполняется, и структура каталогов создается без ошибок.

static int
mkpath(const char *path)
{
  int result = 0;
  struct stat st;
  char *p = NULL, *dir = strdup(path);
  char *tmp = g_malloc0(sizeof(char) * strlen(cache.path) + strlen(dir) + 1);

  dir = dirname(dir);
  tmp = strcpy(tmp, cache.path);
  p = strtok(dir, "/");
  while(p != NULL) {
    tmp = strncat(tmp, "/", 1);
    tmp = strncat(tmp, p, strlen(p));

    if(stat(tmp, &st) == 0) {
      if(S_ISDIR(st.st_mode)) {
        p = strtok(NULL, "/");
        continue;
      }

      result = -ENOTDIR;
      break;
    }

    if(mkdir(tmp, S_IRWXU) == -1) {
      result = -errno;
      break;
    }

    p = strtok(NULL, "/");
  }

  free(tmp);
  free(dir);

  return result;
}

person ben lemasurier    schedule 23.12.2011    source источник


Ответы (3)


Взгляните на справочную страницу для имени каталога: http://linux.die.net/man/3/dirname. «И dirname(), и basename() возвращают указатели на строки с завершающим нулем. (Не передавайте эти указатели в free(3)).» потерян указатель на память, выделенную strdup (память, выделенная strdup, должна передаваться в free).

person Corbin    schedule 23.12.2011

справочная страница для dirname говорит, что вы не должны передавать возвращаемое значение в free(). Тем не менее, это именно то, что вы делаете.

person Oliver Charlesworth    schedule 23.12.2011

согласно справочной странице:

ПРЕДУПРЕЖДЕНИЯ Функция dirname() возвращает указатель на внутреннюю статическую память, которая будет перезаписана последующими вызовами (каждая функция имеет свое отдельное хранилище).

поэтому вы, вероятно, не хотите его освобождать. Я предполагаю, что это отличается от Linux?

person Mike K    schedule 23.12.2011