memcpy caractères de départ supplémentaires

Question

Je vais avoir quelques problèmes en utilisant memcpy en ce que lorsque l'opération est effectuée memcpy je reçois:

« ÍÍWF03-021913.datýýýý« «« «« «« «e »

quand je devrais obtenir:

« WF03-021913.datýýýý« «« «« «« «e »

Je ne sais pas où ces principaux « IĮ » viennent.

code:

Note: lpszFileName = "WF03-021913.dat"

typedef struct { 
    BYTE cbRequestType; 
    BYTE cbFileName; 
    char* szFileName;  
} UFTP_GET_FILE_INFO_REQUEST; 

BOOL Uftp_BuildFileInfoRequest(PUFTP_REQUEST request, LPCTSTR lpszFileName) 
{ 
    UFTP_GET_FILE_INFO_REQUEST *fileInfo; 
    int fileNameLen; 

    if (lpszFileName == NULL) { 
     ASSERT(0); 
     return FALSE; 
    } 

    fileNameLen = strlen(lpszFileName); 
    if (fileNameLen == 0) 
     return FALSE; 

    request->dwRequestSize = sizeof(UFTP_GET_FILE_INFO_REQUEST) - 
          sizeof(void*) + fileNameLen; 
    request->RequestBuffer = malloc(request->dwRequestSize); 
    if (!request->RequestBuffer) { 
     TRACE0("Failed to allocate RequestBuffer"); 
     return FALSE; 
    } 

    fileInfo = (UFTP_GET_FILE_INFO_REQUEST*) request->RequestBuffer; 
    fileInfo->cbRequestType = UFTP_GET_FILE_INFO; 
    fileInfo->cbFileName = fileNameLen; 

    memcpy(&fileInfo->szFileName, lpszFileName, fileNameLen); 
    return TRUE; 
} 

Answer 1

Je ne devine ici, mais je pense que fileInfo->szFileName est un pointeur . Cela signifie que &fileInfo->szFileName est un pointeur vers un pointeur, de sorte que vous copiez dans une autre zone complète de la mémoire.

De même, vous ne copiez pas le caractère de fin requis. Vous avez besoin de fileNameLen + 1 pour cela, à la fois lors de l'allocation et lors de la copie. Si vous voulez vraiment tout dans la mémoire contiguë, vous devriez probablement changer la structure pour finir avec un tableau de caractères de taille zéro (peut ne pas être supporté par votre compilateur, puis utiliser un tableau de taille 1) et utiliser sizeof(UFTP_GET_FILE_INFO_REQUEST) + fileNameLen + 1 comme la taille à allouer. Vous pouvez ensuite utiliser le tableau comme un tableau de chaînes normal.

---

Et si vous fixez ces problèmes, vous avez encore une autre problème: Vous n'initialisez pas le pointeur pour pointer vers la mémoire allouée. Cela signifie qu'il pointera vers une mémoire aléatoire.

Toutes ces erreurs conduiront à undefined comportement, et je dirais que vous êtes chanceux qu'il ne s'est pas écrasé.