我目前正在尝试C
,内存分配和共享内存。我需要帮助,代码是这样的:
#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
#include <string.h>
#include <sys/stat.h>
#include <sys/sem.h>
#include <sys/shm.h>
#include "shared_memory.h"
#include "semaphore.h"
#include "errExit.h"
struct Node {
int ID;
char password[10];
struct Node *next;
};
key_t shmKeyServer = 131;
size_t size = (sizeof(struct Node)) * 100;
int main (int argc, char *argv[]) {
int shmidServer = alloc_shared_memory(shmKeyServer, size);
struct Node *node = (struct Node *)get_shared_memory(shmidServer, 0);
//fill all the structs
for(int i=0;i<100;i++){
node->ID = i;
sprintf(node->password, "%s%i", "campo num:", i);
node->next = node + sizeof(struct Node);
printf("you are on %i cicle \n", i);
node = node->next;
}
return 0;
}
函数alloc_shared_memory
在这里:
int alloc_shared_memory(key_t shmKey, size_t size) {
// get, or create, a shared memory segment
int shmid = shmget(shmKey, size, IPC_CREAT | S_IRUSR | S_IWUSR);
if (shmid == -1)
errExit("shmget failed");
return shmid;
}
get_shared_memory
:
void *get_shared_memory(int shmid, int shmflg) {
// attach the shared memory
void *ptr_sh = shmat(shmid, NULL, shmflg);
if (ptr_sh == (void *)-1)
errExit("shmat failed");
return ptr_sh;
}
问题是在第8次冰柱之后。我得到细分错误。 我认为问题在于内存分配或大小声明。
答案 0 :(得分:1)
问题出在那行:
node->next = node + sizeof(struct Node);
由于typeof(node)
为struct Node *
,因此该语句将node
指针增加sizeof(struct Node) * sizeof(struct Node)
个字节(请参见C中的指针算术)。您想将node
指针增加sizeof(struct Node)
个字节,而不是sizeof(struct Node)
个节点。
您要
node->next = (char*)node + sizeof(struct Node);
// or better:
node->next = (void*)((uintptr_t)(void*)node + sizeof(struct Node));
node->next = (void*)((char*)(void*)node + sizeof(struct Node));
// or
node->next = node + 1;
node->next = &node[1];
修复段错误。
在此行:
sprintf(node->password, "%s%i", "campo num:", i);
发生未定义的行为。 "%s%i", "campo num:", i
正在将12个字节打印到node->password
指针中,该指针仅具有10
个字节的内存:
campo num:1
是11个字符+ 1个字节,用于以零结尾的字符串。同样,对于更大的数字,10
sprintf
将写入13个字节。最好像snprintf
一样使用snprintf(node->password, sizeof(node->password)
来防止缓冲区溢出。您也可以sprintf
返回值int ret = sprintf(..); if (ret > sizeof(node->password)) { err(1, "Overflowed"); }