Problème de mémoire alloué C++

Question

Cela fait longtemps que je n'ai pas programmé en C++, mais j'ai récemment écrit une petite fonction C++ et j'ai un peu de mal. La fonction renvoie une structure, Result, qui contient des chaînes. Je pensais avoir alloué de la mémoire pour les chaînes, mais jsonResult est parfois partiellement écrasé.

//The structs 
    struct Interp { 
     int score; 
     char* sentence; 
     char* jsonResult; 
    }; 

    struct Result { 
     int resultCode; 
     char* errorMessage; 
     Interp interp; 
    }; 

...

//Inside the function 
    Result result; 

    //Store decode 
    const char* jsonResult,* sentence; 
    if (result.resultCode == -1) 
    { 
      LVInterpretation interp = port.GetInterpretation(voiceChannel, 0); 

      result.interp.score = interp.Score(); 
      sentence = interp.InputSentence(); 
      jsonResult = interp.ResultData().Print(SI_FORMAT_ECMA); 
    } 

    //Allocate memory for strings 
    result.interp.jsonResult = new char[strlen(jsonResult) + 1]; 
    strcpy(result.interp.jsonResult, jsonResult); 

    result.interp.sentence = new char[strlen(sentence) + 1]; 
    strcpy(result.interp.sentence, sentence); 

    result.errorMessage = new char[strlen(errorMessage) + 1]; 
    strcpy(result.errorMessage, errorMessage); 

    return result; 

Autres informations: J'observe tout cela derrière le python liant que je l'ai écrit, en utilisant ctypes. Ne pensez pas que cela affecte vraiment quelque chose.

Answer 1

je mettrais de l'argent sur votre problème étant ici:

jsonResult = interp.ResultData().Print(SI_FORMAT_ECMA); 

qui « possède » le tableau de char * retourné par impression()? Peut-être essaie-t-il de renvoyer un pointeur vers une mémoire hors de portée?

exemple:

char* badFunction(void) 
    { 
    char test[100]; 
    strcpy(test,"This is really clever"); // oh, yeah? 
    return test; // returns pointer to data that's out of scope 
    } 

Une autre chose. Affectez des pointeurs NULL à phrase, jsonResult, etc lorsque vous les déclarez. Sinon, vous pourriez finir par strcpy() avec des données non initialisées,

Answer 2

Utilisez std::string. Vous ne le regretterez pas.

Answer 3

Couple de choses:

1. Que signifie "partiellement écrasé" signifie? Comment sais-tu cela? c'est-à-dire quelle est votre sortie attendue par rapport à ce que vous voyez?

2. Il est pas vraiment clair comment result.resultCode est réglé sur -1 (ou si elle est du tout), et si elle est définie, comment la mémoire allouée en se interp.InputSentence() et interp.ResultData().Print(SI_FORMAT_ECMA)? Je vous suggère que votre problème est là

Le reste du code devrait fonctionner aussi longtemps que jsonResult et sentence contiennent des chaînes terminées par null valide.