problème dans l'allocation de la mémoire dans Linux

Question

j'ai signé avec les étoiles les 2 lignes qui créent le problème.

La première ligne alloue de la mémoire pour le fichier journal, qui sera utilisé dans la deuxième ligne signée. à la deuxième rangée signée il y a un problème de faute de segmentation. Ceci est dû au fait que "logfile" n'est pas alloué. Je suis sûr de cela parce que si j'alloue la mémoire dans load() cela fonctionne. Cependant je veux allouer la mémoire dans le constructeur de la classe et non dans la méthode load().

Je ne comprends pas pourquoi ça ne marche pas! Ceci est ma première fois sur Linux et donc peut-être que je fais quelque chose de mal!

Merci, Marco

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, "%s\x00", in_name); 
    sprintf(ip, "%s\x00", in_ip); 
    port = in_port; 
    sprintf(rcon, "%s\x00", in_rcon); 
    sprintf(logfile,"%s\x00", 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; 
} 

Answer 1

1. Je ne vois pas le Destructeur de la classe de serveur. Avez-vous un destructeur qui libère la mémoire?

2. Je ne vois pas le code qui crée et utilise l'objet serveur. Se pourrait-il que vous créiez l'objet serveur mais en fassiez une copie, et le problème se produit parce que vous n'implémentez pas correctement la sémantique de copie?

Answer 2

Je pense que vous essayez de nullterminate in_logfile et in_rcon

Cela ne fonctionnera pas avec printf car printf nécessite une chaîne vide terminé comme arguments à% s en premier lieu.

charptr[known_length] = 0 

au lieu

Answer 3

C'est certainement pas une réponse, mais en C++ développer vous aidera à éviter les problèmes de mémoire que vous obtenez avec votre code C avec classes.

Utilisez des chaînes std ::, puis les copier sera trivial (comparé à sprintf), et il sera beaucoup plus sûr. Utiliser le caractère obsolète * rend les choses très confuses.

Un bon effet secondaire est que vous n'aurez pas besoin de faire une allocation manuelle de la mémoire (avec malloc ou nouveau), et d'éliminer tout risque de fuite de mémoire.