c - 在结构体中存储 void 指针

Question

我在尝试初始化指向 0mq 上下文和套接字的指针结构的代码中遇到段错误。 main 方法中注释掉的代码可以工作，但它仅使用局部变量。我想初始化它们并在结构中传递它们，但我的 google foo 不知道如何正确执行此操作。

#include "zhelpers.h"
#include <stdio.h>
#include <stdlib.h>
#include <zmq.h>

struct publisher{
    void *handle;
    void *context;
};

void init_publisher(struct publisher *p);
void destroy_publisher(struct publisher *p);
void publish(struct publisher *p,char *msg);

void init_publisher(struct publisher *p)
{
    p = (struct publisher *)malloc(sizeof(struct publisher));
    p->context = malloc(sizeof(void *));
    p->handle = malloc(sizeof(void *));
    void *context = zmq_ctx_new();
    void *handle = zmq_socket(context,ZMQ_PUB);
    zmq_bind(handle, "tcp://*:5556");
    zmq_bind(handle, "ipc://feed.ipc");
    p->context = context;
    p->handle = handle;
}

void destroy_publisher(struct publisher *p)
{
    zmq_close(p->handle);
    zmq_ctx_destroy(p->context);
    free(p->handle);
    free(p->context);
    free(p);
}

void publish(struct publisher *p,char *msg)
{
    s_send(p->handle, msg);
}

int main(void)
{
/**
    void *context = zmq_ctx_new();
    void *publisher = zmq_socket(context, ZMQ_PUB);
    int rc = zmq_bind(publisher, "tcp://*:5556");
    assert(rc == 0);
    rc = zmq_bind(publisher, "ipc://weather.ipc");
    assert(rc == 0);
    printf("Started Weather Server...\n");

    srandom((unsigned) time (NULL));
    int zipcode, temperature, relhumidity;
    zipcode = randof(100000);
    temperature = randof (215) - 80;
    relhumidity = randof (50) + 10;

    char update[20];
    sprintf(update, "%05d %d %d", zipcode, temperature, relhumidity);
    s_send(publisher, update);
    zmq_close(publisher);
    zmq_ctx_destroy(context);
*/

    struct publisher *p;
    init_publisher(p);
    printf("Setup pub\n");

    srandom((unsigned) time (NULL));
    int zipcode, temperature, relhumidity;
    zipcode = randof(100000);
    temperature = randof (215) - 80;
    relhumidity = randof (50) + 10;
    char update[20];
    sprintf(update, "%05d %d %d", zipcode, temperature, relhumidity);
    publish(p,update);
    printf("Published Message\n");

    destroy_publisher(p);
    printf("Destroyed publisher\n");
    return 0;
}

Answer 1

此代码中似乎没有任何内容会导致其崩溃。 (假设您知道所有 zmq_... 内容的工作原理。)

如果您准确地告诉我们错误发生的位置，将会有所帮助，但我的猜测是错误发生在这段代码之外。

你看，你将 structpublisher *p 传递给你的 init_publisher() 函数，但随后你在内部为 p 分配内存该方法(这使得传递 p 毫无意义)，然后您就不会返回 p。因此，调用 init_publisher() 的代码可能期望 p 被初始化，但事实并非如此。 p 指向的内存只是在 init_publisher() 函数中本地分配和泄漏。

因此，无需传递 p，只需让函数声明它并返回它即可。

或者，如果调用者已经分配了 p，则不要在 init_publisher() 中再次分配它。

<小时/>

^{另请注意，语句 p->context = malloc(sizeof(void *)); 是不必要的，它们会泄漏少量内存，因为您继续覆盖这些语句结构体成员。}