C语言中适配g_thread_new()的线程函数类型修正问题

你的多线程程序可正常运行，但编译时会抛出函数类型不匹配的警告。以下是你的代码结构及问题分析：

代码结构 #

头文件 main.h #

//=============main.h=============
// Main data structure definition
typedef struct _SPSData SPSData;
struct _SPSData {
  GtkWidget *main_window;
  GtkWidget *menubar1;
  GtkWidget *view;
  GtkWidget *parent;  // Pointer to parent that spawned an error. e.g., data->menubar1, data->main_window, or data->traceroute_window

  char *error_text;  // String to hold error messages.
  char *warning_text;  // String to hold warning messages.

  etc.
};

// Thread function prototype
int ipv4_send (SPSData *);

线程函数 ipv4_send.c #

//==========ipv4_send.c=============
int ipv4_send (SPSData *data)
{ 
  
  some error detection
  if error {
    return (EXIT_FAILURE);
  }
  
  Send ipv4 stuff.

  return (EXIT_SUCCESS);
}

UI回调函数 callbacks.c #

//==========callbacks.c=======
// Send packet(s)
int on_button1_clicked (GtkButton *button1, SPSData *data)
{
  (void) button1;  // This statement suppresses compiler warning about unused parameter.

  GThread *thread;

  // Spawn thread to send IPv4 packet(s).
  thread = g_thread_new ("ipv4_send", (GThreadFunc) ipv4_send, data);
  if (! thread) {
    fprintf (stderr, "Error: Unable to create new thread for ipv4_send() in on_button1_clicked().\n");
    exit (EXIT_FAILURE);
  }

  return (EXIT_SUCCESS);
}

编译警告 #

callbacks.c: In function ‘on_button1_clicked’:
callbacks.c:3846:41: warning: cast between incompatible function types from ‘int (*)(SPSData *)’ {aka ‘int (*)(struct _SPSData *)’} to ‘void * (*)(void *)’ [-Wcast-function-type]
 3846 |     thread = g_thread_new ("ipv4_send", (GThreadFunc) ipv4_send, data);

解决方案 #

你不需要把线程函数改成返回int*（这种方式会引入内存管理问题），正确的做法是让线程函数完全符合GThreadFunc的签名要求：void* (*)(void*)，具体修改如下：

1. 修改线程函数及原型 #

头文件 main.h 中的原型修改：

// Thread function prototype
void* ipv4_send (void *);

ipv4_send.c 中的函数实现修改：

//==========ipv4_send.c=============
void* ipv4_send (void *arg)
{ 
  // 将void*参数转换回SPSData*类型
  SPSData *data = (SPSData*)arg;
  
  // 原有的错误检测逻辑
  if (error_condition) {
    // 用GLib宏将整数状态码转为指针返回
    return GINT_TO_POINTER(EXIT_FAILURE);
  }
  
  // 发送IPv4数据的逻辑
  
  // 成功状态同理转换
  return GINT_TO_POINTER(EXIT_SUCCESS);
}

2. 修改回调函数中的调用 #

去掉强制类型转换，因为现在函数签名完全匹配：

thread = g_thread_new ("ipv4_send", ipv4_send, data);

3. （可选）获取线程返回值 #

如果需要判断线程执行结果，可以在合适的时机调用g_thread_join()，再将返回的指针转回整数：

gpointer thread_result = g_thread_join(thread);
if (GPOINTER_TO_INT(thread_result) == EXIT_FAILURE) {
    // 处理线程执行失败的逻辑
}

说明 #

• GINT_TO_POINTER和GPOINTER_TO_INT是GLib提供的安全宏，用于在整数和指针类型之间转换，避免直接强制转换的潜在风险。
• 不要采用返回int*的方案，因为需要动态分配内存存储状态码，后续还要手动释放，容易引发内存泄漏，完全没必要。

内容的提问来源于stack exchange，提问作者Dave Buchan