avertissement C4172: adresse de retour de la variable locale ou temporaire

Question

Possible en double:
Pointer to local variable

J'ai lu beaucoup d'autres sujets sur ce site sur le même problème sachant que ce serait être commun. Mais je suppose que je suis stupide et ne peux pas trouver la bonne façon de le faire. Donc, je m'excuse pour encore une de ces questions et j'espère que quelqu'un peut me donner une solution simple et/ou une explication.

Voici le code entier:

Main.c

#define WIN32_LEAN_AND_MEAN 

#include <Windows.h> 
#include <stdlib.h> 
#include <tchar.h> 


LPTSTR GetApplicationPath (HINSTANCE Instance); 


int APIENTRY _tWinMain (HINSTANCE Instance, HINSTANCE PreviousInstance, LPTSTR CommandLine, int Show) 
{ 
    LPTSTR sMessage = GetApplicationPath (Instance); 

    MessageBox (
     NULL, 
     sMessage, 
     _T ("Caption!"), 
     MB_OK 
    ); 

    return 0; 
} 


LPTSTR GetApplicationPath (HINSTANCE Instance) 
{ 
    _TCHAR sReturn[MAX_PATH]; 

    GetModuleFileName ((HMODULE) Instance, sReturn, MAX_PATH); 

    return sReturn; 
} 

Answer 1

En ce moment, vous retournez l'adresse d'un tableau automatique (pile). C'est toujours erroné, car dès que la fonction se termine, la durée de vie de cette mémoire est également perdue.

Vous devez utiliser malloc (et free), ou toute autre allocation dynamique. E.g .:

_TCHAR *sReturn = malloc(sizeof(_TCHAR) * MAX_PATH); 

J'ai omis la vérification d'erreur. Plus tard, le code appelant devrait le libérer. Après la MessageBox dans _tWinMain:

free(sMessage); 

Answer 2

L'avertissement dit tout. Dans la fonction GetApplicationPath vous renvoyez sReturn qui est locale à la fonction. Les variables locales cessent d'exister une fois que la fonction retourne et la référence à celles-ci après le retour de la fonction est incorrecte.

Pour remédier à cela, vous pouvez allouer dynamiquement l'espace SOF sReturn comme:

sReturn = malloc(sizeof(_TCHAR) * MAX_PATH); 

Answer 3

Envisager de modifier:

LPTSTR GetApplicationPath (HINSTANCE Instance) 

à quelque chose comme

void GetApplicationPath (HINSTANCE Instance, LPTSTR str) 

Cela permettra d'éliminer la nécessité de revenir une variable locale.

Answer 4

changement

 _TCHAR sReturn[MAX_PATH];

dans votre "GetApplicationPath" à

 static _TCHAR sReturn[MAX_PATH];

Si un seul thread va être appeler GetApplicationPath(), allocation dynamique de cette chaîne est surpuissant. Marquer une variable locale comme static lui alloue de l'espace avec les globals, de sorte que l'espace ne soit pas activé lorsque vous quittez la fonction.