освобождение указателя char в c вызывает segfault

Question

Тема говорит все, вот соответствующий код, написанный в простом файле.

char* fn = NULL; 
SDL_Log("fn pre address is: %p\n",fn); 
fn = get_resource("test.txt"); 
SDL_Log("fn pos address is: %p\n",fn); 
//free(fn); <--this causes seg fault if I uncomment it 

INFO: fn pre address is: 0x0 
INFO: fn pre address is: 0x7fa882501a00 
INFO: fn pos address is: 0x7fa882501a00 

get_resource(const char* filename) { 
    char* string = (char*)calloc(1, sizeof(strlen(res_dir) + strlen(filename))); 
    //i also tried with strncpy 
    //strncpy(string, res_dir, strlen(res_dir)); 
    //strncpy(string + strlen(res_dir), filename, strlen(filename)); 

    memcpy(string, "/usr/test/files", strlen(res_dir)); 
    memcpy(string + strlen("/usr/test/files"), filename, strlen(filename)); 
    string[strlen(filename) + strlen("/usr/test/files")] = '\0'; 
    return string; 
} 

получение такого же поведения. Если я освобожу выделенную строку, я получу ошибку seg. Кроме того, это не всегда случается так странно.

Answer 1

Прежде всего это заявление

char* string = (char*)calloc(1, sizeof(strlen(res_dir) + strlen(filename))); 

неверен. Выражение

sizeof(strlen(res_dir) + strlen(filename)) 

эквивалентно

sizeof(size_t) 

и может быть равным 4 или 8 в зависимости от используемой среды.Более этого выражения

sizeof(strlen(res_dir) + strlen(filename)) 

является невычисленного. То есть функция strlenне называется в этом выражении.

Правильное утверждение может выглядеть

char *string = calloc(strlen(res_dir) + strlen(filename) + 1, sizeof(char)); 

Возьмите в АСоип, что вам нужно также зарезервировать память для завершения ноля.

Однако даже это утверждение неверно. :)

Подумайте, что вы пытаетесь сделать

memcpy(string, "/usr/test/files", strlen(res_dir)); 
memcpy(string + strlen("/usr/test/files"), filename, strlen(filename)); 

Как вы прошли строку "test.txt", то вы получите

"/usr/test/filestest.txt" 

Я думаю, что вы означает следующее:

"/usr/test/files/test.txt" 

Если да, то распределение памяти должно выглядеть

char *string = calloc(strlen(res_dir) + strlen(filename) + 2, sizeof(char)); 

В этом случае вы можете просто написать

strcpy(string, "/usr/test/files"); 
strcat(string, "/"); 
strcat(string, filename); 

и в этом случае заявление

string[strlen(filename) + strlen("/usr/test/files")] = '\0'; 

должны быть удалены.

Answer 2

char* string = (char*)calloc(1, sizeof(strlen(res_dir) + strlen(filename))); 

Эта линия не делает любой смысл, sizeof является оператором для определения размера памяти, необходимый для типа или выражения какого-либо типа. (*)

Что вы хотите это что-то вроде:

Обратите внимание на + 1 там, она оставляет пространство для завершающего 0 байта в строке с стилем. Также обратите внимание на литье указателя void * (как было возвращено malloc() и друзьями в C) не имеет смысла. Это необходимо в C++, но в C void * - это общий указатель типа и неявно преобразуется в любой другой тип указателя данных.

Копаем дальше в ваш код (согласно замечанию алка) ... Думаю, res_dir всего лишь "/usr/test/files"? Здесь может отсутствовать косая черта (в зависимости от содержимого вашего filename), но в целом для конкатенации строк уже существует функция на языке C, объявленная в string.h. Просто напишите что-то простое:

char* string = calloc(1, strlen(res_dir) + strlen(filename) + 1); 
strcpy(string, res_dir); 
strcat(string, filename); 

и все готово. Обратите внимание, что вам не нужен calloc() с этим методом, malloc() все будет в порядке.

---

(*), добавив здесь даже немного больше объяснений: Вы, вероятно, увидеть sizeof довольно часто используется при распределении памяти, по понятным причинам. Вы хотите зарезервировать память для типа char. И вам нужно подсчитать char s, чтобы вписаться в эту память. Таким образом, правильная формула для этого было бы что-то вроде:

malloc((strlen(res_dir) + strlen(filename) + 1) * sizeof(char)) 

Тогда почему оставить из sizeof(char)? Это потому, что размер char фактически равен будет 1. x * 1 все еще x, поэтому нет необходимости использовать sizeof здесь :)