如何分配合适的内存量(c)

时间:2019-05-30 09:22:54

标签: c shared-memory

我目前正在尝试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次冰柱之后。我得到细分错误。 我认为问题在于内存分配或大小声明。

1 个答案:

答案 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"); }