3

I have a large block of memory and I am passing bits of this memory to these functions:

void setBlockSize(char* node, int size) {
    printf("need size: %i\n",size);
    memcpy(node, (void *)&size, sizeof(size));
    printf("set to: %i\n",node); 
}

//written for 4 byte pointer, 32 bit addressing
void setNextPointer(char* node, char* next){
    printf("need ptr: %p\n", next);
    memcpy((node+4), (void*)&next, sizeof(next));
    printf("next: %p, set: %p\n",next, (void*)(node+4));
}

My output is as follows:

need size: 8296
set to: 137666560
need ptr: (nil)
next: (nil), set: 0x834a004
need size: 137666456
set to: 137666560
need ptr: 0xffee4874
next: 0xffee4874, set: 0x834a004
need size: 104
zsh: segmentation fault (core dumped)  ./mallocTest

It appears that the wrong values are being set (I am trying to set a pointer, and an integer. Is this not correct usage for memcpy?

4

1 に答える 1

3

最初の関数では、書き込んだばかりの値を出力するときにポインターを逆参照していません。これを試して:

printf("set to: %i\n", *(int*)node); 

2番目の関数にも同じ問題があります。それ自体node+4ではなく、 pointer 内に格納されているポインター値を出力したい場合:node+4

printf("next: %p, set: %p\n", next, *(void*)(node+4));

に関してだけmemcpy、単一の値を書き込むために使用するのは奇妙です。memcpy次のように回避できます。

*(int*)node = size;
*(char**)(node+4) = next;
于 2013-11-07T03:53:45.130 に答える