我的主要意图是将指针传递给一个结构和一个函数,该函数将分配内存并填充其所有值。回来后我会把它打印在屏幕上。
结构看起来像这样。
struct isolock{
char *name;
unsigned int state;};
typedef struct isolock lock;
请注意,我不能更改此结构的声明。我的主要功能很简单,就像这样。
int main(){
int i = 0;
isolock *lock = NULL;
fillArray(&lock);
//print struct contents
for(i=0;(lock+i)->name;i++){
printf("name = %s status = %u n", (lock+i)->name,(lock+i)->state);
}
}
fillArray函数假设为我的结构分配n+1个内存块,其中n是存在的实际锁数。而最后(n+1)个块将被零填充。因此,当从main打印时,我可以检查这种情况,而不需要担心我分配的结构的长度。
我的问题是fillArray函数。
static const char *array[] = {"SWMGMT_LOCK","OTHER_LOCK"};
void fillArray(isolock **dlock){
int i = 0;
if (*dlock == NULL){
*dlock = (isolock *)malloc((sizeof (*dlock)) * 3); //allocate space for holding 3 struct values
for(i=0;i<2;i++){
(*(dlock) + i)->name = strdup(array[i]);
(*(dlock) + i)->state = (unsigned int)(i+1);
}
(*(dlock) + i)->name = 0; //LINE100
(*(dlock) + i)->state = 0;
}
}
o/p:
name = status = 1
name = OTHER_FP_LOCK status = 2
实际上,这个LINE100造成了问题。它实际上替换了已经填充的结构条目,即这一行使dlock[0]->名称用0填充。而dlock[1]保持不变。
我的gdb日志显示了这样的内容。
所有这些都是在分配和填充值之后获取的日志。
(gdb) p (*(dlock)+0)
$10 = (isolock *) 0x804b008
(gdb) p (*(dlock)+1)
$11 = (isolock *) 0x804b010
(gdb) p (*(dlock)+2) <== Note here 1
$12 = (isolock *) 0x804b018
(gdb) p *(*(dlock)+0)
$13 = {name = 0x804b018 "SWMGMT_LOCK", state = 1} <== Note here 2
(gdb) p *(*(dlock)+1)
$14 = {name = 0x804b028 "OTHER_FP_LOCK", state = 2}
(gdb) n
33 (*(dlock) + i)->state = 0;
(gdb)
35 }
(gdb) p *(*(dlock)+2)
$15 = {name = 0x0, state = 0}
(gdb) p (*(dlock)+2)
$16 = (isolock *) 0x804b018
从注释1和注释2可以清楚地看出,strdup已经返回了malloc已经分配的内存位置,即,对strdup的第一次调用已经返回了dlock[2]的地址。这怎么会发生。因此,(*dlock+2)->name=0导致dlock[0]->name被0填充。
为了容易地解释这个问题,马洛克给我回了三个地址。为了便于理解,例如{100010081010}我已经调用了两次strdup,它返回了{10101018}
这个1010和1018分别存储在lock[0]和lock[1]的char*名称中。
有人能告诉我,我在这段代码中做错了什么吗?或者这是strdup(分配已经分配的内存块)的问题吗
注意:当我将char*name更改为char-name[20],而不是strdup时,我使用了strcpy,它运行得很好。
错误的一个可能原因是您的分配:
*dlock = (isolock *)malloc((sizeof (*dlock)) * 3);
这里,dlock
是指向指针的指针,因此sizeof(*dlock)
是与sizeof(struct isolock)
(或sizeof(**dlock)
)不同的指针的大小。