Указатель строки в C, переписывающий существующий указатель

Question

У меня была очень странная проблема в программе на языке C с перестановкой переменных друг на друга. У меня есть переменная logFilePath, которая определяется следующим образом:

logFilePath = pathToFile(logDirectory,argv[2]); 

Функция pathToFile реализуется как это.

char *pathToFile(char *directory, char *file) { 
    char *filePath = malloc(sizeof(*directory)+sizeof(*file)+10); 
    int i; 
    for (i = 0; directory[i] != '\0'; i++) { 
    if (directory[i+1] == '\0' && directory[i] != '/') strcat(directory,"/"); 
    } 
    strcat(filePath, directory); 
    strcat(filePath, file); 
    return filePath; 
} 

По какой-то причине logFilePath модифицируется полностью несвязанной функцией. Функция, в которой он изменяется, называется так

line = getIdentifierFromLine(line); 

И реализация.

char *getIdentifierFromLine(char *line) { 
    char *ident = malloc(sizeof(char)*30); 
    int i = 0; 
    printf("%s\n", ident); 
    while (line[i] != ';') { 
    ident[i] = line[i]; 
    i++; 
    } 
    return ident; 
} 

Проблема заключается в том, что logFilePaths часть, кажется, тот же идент, а именно половина после первых 15 символов. Идентификатор инициализируется части после первых 15 символов logFileDirectory и изменения значения каждого символа в идентификаторе изменяет его в logFileDirectory. Что здесь происходит?

Answer 1

sizeof не делает проверку длины строки. Это:

char *filePath = malloc(sizeof(*directory)+sizeof(*file)+10); 

неправ. sizeof(*directory) дает размер символа (обычно 1).

Что вы хотите, это:

char *filePath = malloc(strlen(directory)+strlen(directory)+2); 

2 экстра достаточно, один для дополнительного / и один для нулевого завершающего символа.

Из-за этого вы не выделяете достаточное количество памяти для своего пути к файлу, что приводит к неопределенному поведению. В вашем случае перезапись буферов. Это может также привести к segfaults или другим странным вещам (у меня было это когда-то, когда действительный фрагмент кода был прерван, потому что я допустил ошибку где-то в другом месте.)

Answer 2

Стандартный метод ведения журнала (по моему опыту) заключается в использовании потоков stdout или stderr. В C++, чтобы использовать их вам нужно будет включить iostream, и использовать, как показано ниже:

#include <iostream> 

int main(int argc, char* argv[]) 
{ 
    using std::cout; 
    using std::cerr; 
    using std::endl; 

    cout << "Output message" << endl; 
    cerr << "Error message" << endl; 
} 

Это, однако, только достигает печать на этих выходах, которые обычно заканчиваются в терминале. Если вы хотите использовать эти стандартные методы потока (которые вполне читаемы) для вывода в файл, вам нужно как-то перенаправить ваш вывод. Один из способов сделать это - использовать функцию freopen, предоставляемую cstdio. Это означает, что файл открыт и перемещает данный поток в этот файл. См. Здесь документацию. Примером может быть:

#include <iostream> 
#include <cstdio> 

int main(int argc, char* argv[]) 
{ 
    using namespace std; 
    freopen("output.txt", "w", stdout); 
    freopen("error.txt", "w", stderr); 

    cout << "Output message" << endl; 
    cerr << "Error message" << endl; 
} 

Answer 3

Проблема заключается в вычислении размера, когда вы выделяете место для filePath:

char *filePath = malloc(sizeof(*directory)+sizeof(*file)+10); 

Здесь directory имеет тип char*, так *directory имеет тип char и sizeof(*directory) является 1. То же самое происходит и с sizeof(*file) и выделяется только 12 байт памяти.

Если этого недостаточно места, путь записывается за конец буфера в пространство, которое позже заканчивается на ident в этой другой функции. Затем запись в *indent изменяет ту же память, которая занята в конце filePath.

Вместо sizeof вы должны использовать strlen() для определения длины входных строк:

char *filePath = malloc(strlen(directory)+strlen(file)+10);