pointeur et memoire partagée

**troumar** · 13/09/2012, 12h35

Bjr, j'ai une methode en C qui attribue une adresse memoire à une structure de donnees qu'elle a reçu en parametre. et a la sortie de cette methode, je dois nourrir les champs de cette stucture. Je recois une erreur:

Program received signal SIGSEGV, Segmentation fault.
0x000000000041519d in SC_ShmLoadBankInfo () at SCPubFunc.pc:102
SC_ShmGetStru(g_stConf.iShmID, MODE_BANK_ARRAY, i, (void *)&pstBank);
102 pstBank->iBankID = iBankID;

voici la methode qui attribue l'adresse memoire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
int SC_ShmGetStru(int iShmID, int iMode, int iID, void **pAddr)
{
    int i, iOffset, iStruSize;
    ST_SC_BankInfo *pstBank = NULL;
    ST_SC_SvcInfo *pstSvc = NULL;
    void *pBase;
 
    /* get shm base addr (attach) */
    if ((int)(*pBase = (void*)shmat(iShmID, (void*)0, 0)) == -1)
    {
        sprintf(g_caMsg, "[Fail] ShmGetStru, attach shm");
        ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
        return -1;
    }
 
    switch (iMode)
    {
    case MODE_CONFIG:
        *pAddr = (ST_SC_Config *) pBase;
        return 0;
    case MODE_BANK_ARRAY:
        iOffset = sizeof(ST_SC_Config);
        iStruSize = sizeof(ST_SC_BankInfo);
        if (iID > BANK_NUM)
        {
            sprintf(g_caMsg, "[Fail] ShmGetStru, illegal bank array %d", iID);
            ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
            shmdt(pBase);
            return -1;
        }
        *pAddr = (ST_SC_BankInfo *) (pBase + iOffset + sizeof(ST_SC_BankInfo) * iID);
        return 0;
    case MODE_BANK_VALUE:
        iOffset = sizeof(ST_SC_Config);
        iStruSize = sizeof(ST_SC_BankInfo);
        for (i=0; i<BANK_NUM; i++)
        {
            pstBank = (ST_SC_BankInfo *) (pBase + iOffset + iStruSize * i);
            if (pstBank->iBankID == iID)
            {
                *pAddr = pstBank;
                return 0;
            }
        }
        sprintf(g_caMsg, "[Fail] ShmGetStru, bank value not found:%d", iID);
        ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
        shmdt(pBase);
        return -1;
    case MODE_SVC_ARRAY:
        if (iID > g_stConf.iSvcNum)
        {
            sprintf(g_caMsg, "[Fail] ShmGetStru, illegal svc array %d", iID);
            ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
            shmdt(pBase);
            return -1;
        }
        iOffset = sizeof(ST_SC_Config) + sizeof(ST_SC_BankInfo) * BANK_NUM;
        iStruSize = sizeof(ST_SC_SvcInfo);
        pstSvc = (ST_SC_SvcInfo *) (pBase + iOffset + iStruSize * iID);
        *pAddr = pstSvc;
        break;
    case MODE_SVC_VALUE:
        iOffset = sizeof(ST_SC_Config) + sizeof(ST_SC_BankInfo) * BANK_NUM;
        iStruSize = sizeof(ST_SC_SvcInfo);
        for (i=0; i<g_stConf.iSvcNum; i++)
        {
            pstSvc = (ST_SC_SvcInfo *) (pBase + iOffset + iStruSize * i);
            if (pstSvc->iSvcID == iID)
            {
                *pAddr = pstSvc;
                return 0;
            }
        }
        sprintf(g_caMsg, "[Fail] ShmGetStru, svc value not found:%d", iID);
        ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
        shmdt(pBase);
        return -1;
    default:
        sprintf(g_caMsg, "[Fail] ShmGetStru, invalide Mode:%d", iMode);
        ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
        shmdt(pBase);
        return -1;
    }
 
    return 0;
}

La partie du code où le programme plante:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
SC_ShmGetStru(g_stConf.iShmID, MODE_BANK_ARRAY, i, (void *)&pstBank);
pstBank->iBankID = iBankID;
strcpy(pstBank->caDBUsr, caDBUsr);
strcpy(pstBank->caBankName, caBankName);
strcpy(pstBank->caTax, caTax);
strcpy(pstBank->caBin, caBin);
SC_ShmDettach(pstBank);

Merci pour votre aide.

**Neckara** · 13/09/2012, 12h54

Bonjour,

j'ai une methode en C qui attribue une adresse memoire à une structure de donnees qu'elle a reçu en parametre

Ce n'est pas possible en C.
Tu peux déplacer (copie + suppression de l'originale) ou copier une structure mais pas lui "attribuer une adresse mémoire".

Pour ton segfault, je suppose que tu essaye de copier ta structure en oubliant de faire un malloc ?

J'ai vite survolé ton code mais sans les balises codes, il est illisible.

**troumar** · 13/09/2012, 13h12

je viens de mettre les balises. Ce que j'aimerais savoir pourquoi cela plante a ce niveau. Hors je souligne ce code marchait correctement, et depuis que nous sommes passé en 64 bits, nous rencontrons ce probleme.

Program received signal SIGSEGV, Segmentation fault.
0x000000000041519d in SC_ShmLoadBankInfo () at SCPubFunc.pc:102
SC_ShmGetStru(g_stConf.iShmID, MODE_BANK_ARRAY, i, (void *)&pstBank);
102 pstBank->iBankID = iBankID;

voici la methode qui attribue l'adresse memoire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
 
int SC_ShmGetStru(int iShmID, int iMode, int iID, void **pAddr)
{
int i, iOffset, iStruSize;
ST_SC_BankInfo *pstBank = NULL;
ST_SC_SvcInfo *pstSvc = NULL;
void *pBase;
 
/* get shm base addr (attach) */
if ((int)(*pBase = (void*)shmat(iShmID, (void*)0, 0)) == -1)
{
sprintf(g_caMsg, "[Fail] ShmGetStru, attach shm");
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
return -1;
}
 
switch (iMode)
{
case MODE_CONFIG:
*pAddr = (ST_SC_Config *) pBase;
return 0;
case MODE_BANK_ARRAY:
iOffset = sizeof(ST_SC_Config);
iStruSize = sizeof(ST_SC_BankInfo);
if (iID > BANK_NUM)
{
sprintf(g_caMsg, "[Fail] ShmGetStru, illegal bank array %d", iID);
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
shmdt(pBase);
return -1;
}
*pAddr = (ST_SC_BankInfo *) (pBase + iOffset + sizeof(ST_SC_BankInfo) * iID);
return 0;
case MODE_BANK_VALUE:
iOffset = sizeof(ST_SC_Config);
iStruSize = sizeof(ST_SC_BankInfo);
for (i=0; i<BANK_NUM; i++)
{
pstBank = (ST_SC_BankInfo *) (pBase + iOffset + iStruSize * i);
if (pstBank->iBankID == iID)
{
*pAddr = pstBank;
return 0;
}
}
sprintf(g_caMsg, "[Fail] ShmGetStru, bank value not found:%d", iID);
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
shmdt(pBase);
return -1;
case MODE_SVC_ARRAY:
if (iID > g_stConf.iSvcNum)
{
sprintf(g_caMsg, "[Fail] ShmGetStru, illegal svc array %d", iID);
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
shmdt(pBase);
return -1;
}
iOffset = sizeof(ST_SC_Config) + sizeof(ST_SC_BankInfo) * BANK_NUM;
iStruSize = sizeof(ST_SC_SvcInfo);
pstSvc = (ST_SC_SvcInfo *) (pBase + iOffset + iStruSize * iID);
*pAddr = pstSvc;
break;
case MODE_SVC_VALUE:
iOffset = sizeof(ST_SC_Config) + sizeof(ST_SC_BankInfo) * BANK_NUM;
iStruSize = sizeof(ST_SC_SvcInfo);
for (i=0; i<g_stConf.iSvcNum; i++)
{
pstSvc = (ST_SC_SvcInfo *) (pBase + iOffset + iStruSize * i);
if (pstSvc->iSvcID == iID)
{
*pAddr = pstSvc;
return 0;
}
}
sprintf(g_caMsg, "[Fail] ShmGetStru, svc value not found:%d", iID);
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
shmdt(pBase);
return -1;
default:
sprintf(g_caMsg, "[Fail] ShmGetStru, invalide Mode:%d", iMode);
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
shmdt(pBase);
return -1;
}
 
return 0;
}

La partie du code où le programme plante:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
SC_ShmGetStru(g_stConf.iShmID, MODE_BANK_ARRAY, i, (void *)&pstBank);
pstBank->iBankID = iBankID;
strcpy(pstBank->caDBUsr, caDBUsr);
strcpy(pstBank->caBankName, caBankName);
strcpy(pstBank->caTax, caTax);
strcpy(pstBank->caBin, caBin);
SC_ShmDettach(pstBank);
......

Merci pour votre aide.

**Neckara** · 13/09/2012, 13h20

Pour mettre la balise code il faut faire comme sur la petite image :

avec des crochets.

EDIT : le temps de poster vous avez trouver comment mettre la balise code^^

Peux-tu mettre un petit commentaire à la ligne exacte où le programme plante?

Sinon, soit le pointeur sur ta structure ne pointe pas sur une zone allouée soit les pointeurs sur char contenu dans ta structure ne pointent pas sur une zone alloué assez grande.
Si la taille des chaînes de caractère varie, essaye plutôt d'utiliser strncpy.
Sinon un coup de Valgrind t'aidera peut-être.

**troumar** · 13/09/2012, 13h43

voici ou le programme plante:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
SC_ShmGetStru(g_stConf.iShmID, MODE_BANK_ARRAY, i, (void *)&pstBank);
pstBank->iBankID = iBankID;  /* voici ou le prog plante*/
strcpy(pstBank->caDBUsr, caDBUsr);
strcpy(pstBank->caBankName, caBankName);
strcpy(pstBank->caTax, caTax);
strcpy(pstBank->caBin, caBin);
SC_ShmDettach(pstBank);
......

**Neckara** · 13/09/2012, 14h13

pstBank ne pointe donc pas dans une zone mémoire allouée.

SC_ShmGetStru doit donc ne pas donner une valeur correcte à pstBank.
Essayez de voir quelle affectation conduit à une erreur erronée.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

*pAddr = (ST_SC_BankInfo *) (pBase + iOffset + sizeof(ST_SC_BankInfo) * iID);

Vous avez l'air de faire pas mal de calculs, attention, la taille d'une structure est égale ou supérieur à la somme des tailles de se qu'elle contient.

**Médinoc** · 14/09/2012, 10h09

J'ai l'impression que troumar retourne une adresse pointant directement dans la mémoire partagée, alors qu'il fait un shmdt dans la fonction.
Cela n'est-il pas équivalent à ce genre de bêtise?

Code C :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
void* f()
{
	void *p = malloc(1);
	free(p);
	return p;
}

**Neckara** · 14/09/2012, 12h35

Envoyé par Médinoc

J'ai l'impression que troumar retourne une adresse pointant directement dans la mémoire partagée, alors qu'il fait un shmdt dans la fonction.
Cela n'est-il pas équivalent à ce genre de bêtise?

Code C :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
void* f()
{
	void *p = malloc(1);
	free(p);
	return p;
}

Il ne fait le shmdt que s'il a une erreur et dans ce cas, il devrait avoir un message d'erreur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
sprintf(g_caMsg, "[Fail] ShmGetStru, illegal bank array %d", iID);
ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
shmdt(pBase);

**Médinoc** · 14/09/2012, 13h33

Ah OK.

**diogene** · 15/09/2012, 12h37

Le plantage a lieu avec pour case MODE_BANK_ARRAY. Le code exécuté dans ce cas est, en supposant qu'aucune des erreurs détectées par le code ne se produise, et en supprimant des cast abusifs et inutiles :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
int SC_ShmGetStru(int iShmID, int iMode, int iID, void **pAddr) //iMode ==MODE_BANK_ARRAY
{
   int i, iOffset, iStruSize;
   ST_SC_BankInfo *pstBank = NULL;
   ST_SC_SvcInfo *pstSvc = NULL;
   void *pBase;
 
   pBase = shmat(iShmID, NULL, 0);
   iOffset = sizeof(ST_SC_Config);
   iStruSize = sizeof(ST_SC_BankInfo);
   *pAddr = pBase + iOffset + sizeof(ST_SC_BankInfo) * iID;
   return 0;
}

Je suis étonné du calcul de pBase + iOffset + sizeof(ST_SC_BankInfo) * iID avec un pbase de type void* : la taille de l'objet pointé est inconnue.
Vu la présence d'un sizeof dans ce calcul, j'opterai pour :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 *pAddr = (char*)pBase + iOffset + sizeof(ST_SC_BankInfo) * iID;

Quand au if :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if ( (pBase = shmat(iShmID, NULL, 0)) == (void*)-1) ...

me semble logique

pointeur et memoire partagée

C

Vue hybride

Discussions similaires

Partager

Partager