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.

**troumar** · 13/09/2012, 14h47

bon ensuite je ne sais pas si ça peut aider à la resolution mais lors de la compilation j'ai un warning a ce niveau:

SCPub.c: In function SC_ShmGetStru:
SCPub.c:454: attention : transtypage vers un pointeur depuis un entier de taille diffÃ©rente
SCPub.c:454: attention : transtypage d'un pointeur vers un entier de taille diffÃ©rente.

la ligne concernée est la suivante:

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
 
 
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) /* la ligne du warning*/
    {
        sprintf(g_caMsg, "[Fail] ShmGetStru, attach shm");
        ErrLog(TRACE_LEVEL5, g_caMsg, RPT_TO_LOG, 0, 0);
        return -1;
    }
......

je ne comprends pas bien......

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

Je me vois dans l'obligation de sortir la tronçonneuse

Il ne faut jamais ignorer les insultes du compilateur !

EDIT :

Code :

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

1
2
3
4
5
void *pBase; //pointeur sur un "void", sur n'importe quoi en fait.
 
if ((int)(*pBase = (void*)shmat(iShmID, (void*)0, 0)) == -1)
(int)(*pBase); // je doute très fortement que tu aie le droit de faire cela.
*pBase = (void *).... ;//idem

**Bktero** · 13/09/2012, 14h59

Il est interdit de déréférencer un pointeur de type void* sans cast préalable.

Je suis étonné que ton compilateur ne te renvoie pas une erreur pour cela.

**troumar** · 13/09/2012, 15h25

je viens de corrigé cela et voici:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
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;
    }
..........

Mais j'ai tjrs le warning en question..

**Neckara** · 13/09/2012, 15h38

Déjà généralement on met NULL ( ou nullptr C99 ?) et non (void *) 0.

Ensuite, un pointeur invalide vaut NULL et non -1.
Une adresse est codée comme un unsigned int il me semble (mais ce n'est pas un unsigned int).
Donc une adresse "valant -1" c'est : 0xFFFFFFFF (4 294 967 295) ce qui peut être une adresse valide.

Le warning vient de la conversion d'un entier signé vers un entier non-signé de même taille.
Ainsi la moitié des valeurs de l'entier signé sont plus grand que la valeur maximale de l'entier signé.
Elles sont donc converties en valeur négatives.

Sinon quel est le prototype de shmat ?

**Bktero** · 13/09/2012, 15h58

http://man.developpez.com/man2/shmat.2.php

Code :

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

void *shmat ( int shmid, const void * shmaddr, int shmflg);

Je présume.

**Neckara** · 13/09/2012, 16h10

Envoyé par Bktero

http://man.developpez.com/man2/shmat.2.php

Code :

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

void *shmat ( int shmid, const void * shmaddr, int shmflg);

Je présume.

D'accord, je pensais que c'était une fonction à lui.
Théoriquement ceci ne devrait pas faire de warning :

Code :

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

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

Par contre, je suis étonné qu'ils fassent retourner -1 et non NULL en cas d'erreur.

**troumar** · 13/09/2012, 16h53

Code :

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

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

Erreur obtenu: comparaison entre un entier et un pointeur.

**DrDarko** · 13/09/2012, 17h16

Normal ton pBase est un pointeur et tu le compare à -1 qui est un int.

Après je comprend pas trop vu que la fonction doit retourner un pointeur en cas de succès et si elle fail elle retourne -1... Je suis plutôt d'accord avec Neckara la fonction en cas d’échec devrait retourner NULL...

**troumar** · 13/09/2012, 17h26

ok je remplace -1 par NULL, maintenant quand je dois utiliser ma structure pstBank a la sortie de la methode :SC_ShmGetStru, j'ai un plantage.
le plantage est le suivant:
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 ligne en erreur:

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);
......

j'ai comme l'impression que l'addresse memoire partagée n'est pas valide. Alors ce code marchait tres bien en sur un systeme 32 bit. Depuis que nous sommes passé en 64 bits nous constatons cela.....

**Neckara** · 13/09/2012, 17h29

Ceci ne devrait pas mettre le warning :

Code :

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

1
2
3
4
5
pBase = (void*)shmat(iShmID, NULL, 0);
if (!  (pBase + 1) ) //ou pBase == (void *)0u - 1  
{
     //erreur
}

EDIT :

Code :

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

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

~~Pourquoi + iOffset?~~ J'ai rien dit^^
Es-tu sûr que tu alloues bien plus de :

Code :

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

sizeof(ST_SC_Config) + (BANK_NUM + 1)*sizeof(ST_SC_BankInfo)

octets ?
Ou ne serait-ce pas plutôt :

Code :

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

1
2
3
4
5
6
7
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;
}

?

Sinon à moins qu'un autre thread provoque l'erreur ou que la bibliothèque que tu utilises soit compilée en 32bit et non en 64bits, je ne vois pas.

**Bktero** · 14/09/2012, 09h08

En regardant d'autres pages de manuel que celle de Developpez, la fonction semble renvoyer (void*)-1 et non -1 directement :
http://man.cx/shmat%282%29/fr
http://linux.die.net/man/2/shmat

Je trouve aussi cela très étrange mais bon.....

**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);

pointeur et memoire partagée

C

Discussions similaires

Partager

Partager