【问题标题】:Passing void* reference for arbitrary structure storage into functions将任意结构存储的 void* 引用传递给函数
【发布时间】:2020-04-08 09:52:04
【问题描述】:

我有一个用于与 FPGA 寄存器和内存通信的 cli 程序。根据命令,您可能会返回一个 32 位寄存器(可能包含多个字段),也可能是 128 字节。所以我想做的是全局 malloc 128 字节来保存返回的结构,或者保存一个结构以准备将其写入 FPGA。 (参见 void *entry_top。

在下面的示例中,我有一个单数 Get 命令在尝试打印所获得字段的值时失败并出现分段错误。在 GDB 中,我可以看到 entry_top 的地址似乎实际上并没有通过引用传递到 io_get_entry 函数中,这是本意。

我试图通过使用 (host_global_size_entry_t*) 的强制转换为条目分配一个测试值 0x12345678 来让事情变得更简单一些。在这个测试程序中,在主级别,我相信我应该能够将 void * 转换为 (host_global_size_entry_t *) 并正确读取 entry_top 中的设置值。

不是 100% 确定,但问题似乎是我的内存引用在使用 &entry_top 时被错误地传递。

谁能评论我在将 void* entry_top 引用传递给多级函数时做错了什么?

using namespace std;

#include "stdint.h"
#include <stdlib.h>
#include <list>
#include <string.h>
#include <string>
#include <iostream>


#ifndef __KRISTEN_TYPES_H__
#define __KRISTEN_TYPES_H__
#include <stdint.h>

#ifdef __cplusplus
extern "C"
{
#endif /* __cplusplus */
#ifdef PACKED
#undef PACKED
#endif /* PACKED */
#define PACKED __attribute__ ((packed))
#ifdef ALIGNED
#undef ALIGNED
#endif /* ALIGNED */
#define ALIGNED __attribute__ ((aligned (4)))
#ifdef __cplusplus
}
#endif /* __cplusplus */


typedef enum _return_t  
{                
    /* Common codes */  
    func_ok = 0,        
    func_failed,        
    func_invalid,       
    func_not_done       
} return_t;              

#endif // __KRISTEN_TYPES_H__


 uint32_t host_global_size_offset[1] = {53300};


 void *entry_top = malloc(32*sizeof(uint32_t));
 uint32_t *field_size = (uint32_t*)malloc(sizeof(uint32_t));
 uint32_t* result=(uint32_t*)malloc(64*sizeof(uint32_t));

struct _host_global_size_entry_t {
   union {
      struct {
         uint32_t global_size_field        : 32 ; // A generic bit field for this register.
      } PACKED; 
      uint8_t entry[5];
   } PACKED;
   uint8_t alignment_correction[3];
} PACKED;
typedef struct _host_global_size_entry_t __attribute__ ((aligned (4))) host_global_size_entry_t;


return_t io_get_entry(uint32_t index, void *entry, uint32_t base, uint8_t  length) {
   uint32_t local_addr = base + (index * length);
   ((host_global_size_entry_t*) entry)->global_size_field = 0x12345678;  // Assign a value that we can see in Main'
   // The next line shows that the value was 0x12345678 succesfully
   printf("io_get_entry local_addr = %d, length = %d, entry->global_sizefield = 0x%X\n", local_addr, length, ((host_global_size_entry_t*) entry)->global_size_field);
   return func_ok;
}


return_t host_global_size_read_entry(uint32_t index, void *entry, uint8_t instance) {
   return_t result;
   uint32_t base;
   uint8_t  length;
   base = host_global_size_offset[instance];
   length = sizeof(host_global_size_entry_t);
   result = io_get_entry(index, &entry, base, length);
   // The next printf causes a segmentation fault.  GDB says that &entry here is not the same as &entry in io_get_entry 
   printf("host_global_size_read_entry = 0x%X\n", ((host_global_size_entry_t*)entry)->global_size_field);

   return result;
}

return_t host_global_size_read_field(char *field, uint32_t *field_size, void *result, void *entry) {
//host_global_size_entry_t *entry;
   uint32_t field_s = 0;
   if (strcmp("global_size_field", field) == 0) {
      field_s = 4;
      *(uint32_t *)result = ((host_global_size_entry_t*)entry)->global_size_field; 
   } else   if (field_size != NULL) *field_size = field_s;
      *field_size = field_s;
   return func_ok;
}



int main (void) {
  uint8_t instance = 0;
  uint32_t index = 0;
  // Pass the address of entry_top global
  host_global_size_read_entry(index, &entry_top, instance); // Read entry.
  // Cast the result to the structure we expect to be comming back.  
  printf("main: entry_top->global_sizefield = 0x%X\n", ((host_global_size_entry_t*) entry_top)->global_size_field);
  return 0;
}

【问题讨论】:

  • 您定义了几个保留标识符,例如__KRISTEN_TYPES_H___return_t_host_global_size_entry_t。程序的行为将是未定义的。你应该重命名它们。
  • 不相关:使用new uint32_t[32] 而不是malloc

标签: c++ reference pass-by-reference


【解决方案1】:

mainhost_global_size_read_entry 中,您都使用entry_top地址(分别是entry) 这是 void **,但 C++ 会默默地将其转换为 void * 而不会引起任何投诉。

相反,您应该按值传递entry_top,以便host_global_size_read_entryio_get_entry 接收entry_top 的内容,即指向已分配内存的指针。

【讨论】:

  • 那行得通。更改 host_global_size_read_entry(index, &entry_top, instance);到 host_global_size_read_entry(index, entry_top, instance);解决了这个问题。有点沮丧,我看不到那里发生了什么。感谢您的帮助。
猜你喜欢
  • 2013-10-19
  • 2015-12-23
  • 2013-12-12
  • 1970-01-01
  • 2012-05-23
  • 2012-11-08
  • 2012-05-15
  • 2021-12-27
  • 1970-01-01
相关资源
最近更新 更多