Блог

я подписал звездочками 2 строки, которые создают проблему.

первая строка выделяет память для файла журнала, который будет использоваться во второй подписанной строке. во второй строке со знаком возникает проблема с ошибкой сегментации. Это вызвано тем фактом, что «logfile» не выделен. Я уверен в этом, потому что, если я выделяю память в load (), это работает. Однако я хочу выделить память в конструкторе класса, а не в методе load().

Я не могу понять, почему это не работает! Я впервые в Linux, и поэтому, возможно, я делаю что-то не так!

Спасибо, Марко

     server::server(){
    port = 0;
    serverup = 0;
    loaded = 0;
    logfile = (char *) malloc(SERVER_PATHS_SIZE*sizeof(char)); //**************************** 
}

int server::load(int in_id, char *in_name, char *in_ip, int in_port,
                 char *in_rcon, char *in_logfile){

    int err;

    sprintf(name, "%sx00", in_name);
    sprintf(ip, "%sx00", in_ip);
    port = in_port;
    sprintf(rcon, "%sx00", in_rcon);
    sprintf(logfile,"%sx00", in_logfile); //**********************************

    err = urt.set(ip, port, rcon);
    if(err < 1){
        printf("server::load(): error from urt.set()n");
        return 0;
    }

    printf("server::load(): server %d loaded!n", id);
    loaded = 1;

    return 1;
}
  

Я думаю, вы пытаетесь выполнить nullterminate in_logfile и in_rcon

Это не будет работать с printf, потому что для printf в первую очередь требуется строка с нулевым завершением в качестве аргументов для %s.

 charptr[known_length] = 0
  

вместо

Это определенно не ответ, но разработка на C поможет вам избежать проблем с памятью, которые возникают при использовании кода C-with-classes.

Используйте std::strings, тогда их копирование будет тривиальным (по сравнению со sprintf), и это будет намного безопаснее. Использование устаревшего символа char * приводит к путанице.

Приятным побочным эффектом является то, что вам не нужно будет вручную выделять память (с помощью malloc или new) и устранять любой риск утечки памяти.

1. Я не вижу деструктора класса server. У вас есть деструктор, который освобождает память?

2. Я не вижу кода, который создает и использует серверный объект. Может ли быть так, что вы создаете объект server, но затем делаете его копию, и проблема возникает из-за того, что вы неправильно реализуете семантику копирования?