struct和malloc（）/ free（）问题

|| 我正在尝试将双向链接列表映射到GUI。我基本上为列表中的每个节点创建一个按钮结构，将节点参数映射到按钮参数，然后在屏幕上显示它们。我可以添加很多按钮，超过500个，它们似乎保持其唯一数据并且不重叠。但是，当我仅删除一个按钮时，下一个malloc（）函数将无法创建新按钮，并且只能删除。如果我摆脱了free（）函数，则malloc（）不再失败。所以，我的问题是我是否正确创建和删除这些结构？ 这些结构取自我为在特定硬件上工作而修改的GUI库。 
typedef struct
{
    obj_t    obj;     /**< object structure */
    string_t label;   /**< button label, NULL if no label */
    color_t  color;   /**< color of the button */
} button_t;

typedef struct obj_t
{
    int                   x;                 /**< x position, relative to parent */
    int                   y;                 /**< y position, relative to parent */
    unsigned              width;             /**< component widht */
    unsigned              height;            /**< component height */
    draw_t                draw;              /**< function to draw the component */
    handler_t             handler;           /**< function to handle mouse-input events */
    action_t              action;            /**< function to handle user action */
    struct obj_t         *parent;            /**< parent of the component */
    unsigned              agui_index;        /**< agui structure index */
    BOOL                  enabled;           /**< if set, component is enabled */
    BOOL                  visible;           /**<  */
    BOOL                  invalidated;       /**< if set, draw this component */
    BOOL                  invalidated_child; /**< children need to be redrawn */
    BOOL                  selected;           /**< component is selected */
    BOOL                  pressed;           /**< component is pressed */
    uintptr_t             tag;               /**< tag for general use */
} obj_t;

typedef struct
{
    int           x;         /**< x position */
    int           y;         /**< y position  */
    char         *text;      /**< string text */
    const font_t *font;      /**< string-text font */
    color_t       color;     /**< string-text color */
    fontstyle_t   fontstyle; /**< string-text font style */
    align_t       align;     /**< alignment with respect to x and y position */
} string_t;
这些是使用malloc（）的代码片段： 
char *step_name = (char*)malloc(20*sizeof(char));
if(step_name == NULL)
    return -1;

sprintf(step_name,\"STEP %d\",curr_job_recipe->curr_step->step_num);

obj_t *obj_step = (obj_t*) malloc(sizeof(obj_t));
if(obj_step == NULL)
{
    free(step_name);
    return -1;
}

string_t *label_step = (string_t*) malloc(sizeof(string_t));
if(label_step == NULL)
{
    free(step_name);
    free(obj_step);
    return -1;
}

button_t *newstep_button =(button_t*) malloc(sizeof(button_t));
if(newstep_button == NULL)
{
    free(step_name);
    free(obj_step);
    free(label_step);
    return -1;
}

obj_t **objects; // This is a parameter to the function I\'m simplifying to save sanity 
objects[curr_index] = &newstep_button->obj;
obj_step->x = 2;
obj_step->y = objects[curr_index-1]->y+BUTTON_HEIGHT+1;
obj_step->width = 316;
obj_step->height = 60;
obj_step->draw = button_draw;
obj_step->handler = button_handler;
obj_step->parent = AGUI_HANDLE(editrecipeform);
obj_step->agui_index = 0;
obj_step->action = editrecipeform_btn5_action;
obj_step->visible = TRUE;
obj_step->enabled = TRUE;
obj_step->selected = FALSE;
obj_step->pressed = TRUE;
obj_step->invalidated = TRUE;
label_step->x = 0;
label_step->y = 0;
label_step->text = step_name;
label_step->font = &helveticaneueltstdltext18_2BPP;
label_step->color = BLACK;
label_step->fontstyle = FS_NONE;
label_step->align = ALIGN_CENTRE;
newstep_button->obj = *obj_step;
newstep_button->label = *label_step;
newstep_button->color = RED;
然后，当用户在按钮上选择删除时，将执行以下操作。 
button_t *newstep_button = (button_t*) objects[i];
obj_t *obj_step = (obj_t*) &newstep_button->obj;                
string_t *label_step = (string_t*) &newstep_button->label;
free(label_step->text);
free(label_step);
free(obj_step);
free(newstep_button);
编辑：添加了一些我在malloc（）代码空间区域中遗漏的初始化代码     
2020-01-24 4 条评论
分享

没有找到相关结果

    已邀请:

    吠强祷豪硅 

    obj_t *obj_step = (obj_t*) &newstep_button->obj;
free(obj_step);
    在这里，您尝试释放一个非指针结构字段。您不应该这样做：当您调用ѭ4deal时，整个结构的内存将被释放，并且不需要分别取消分配字段。 
    label
    字段也是如此。 如果您的结构体具有用
    malloc
    分配的指针字段，则需要先释放它们，然后再释放结构体本身。 关于分配的
    obj_step
    和
    label_step
    变量，需要释放它们，但是从发布它们存储值的代码中尚不清楚。如果未在任何地方使用它们，则可以删除这两个
    malloc
    。 编辑：您的初始化代码基本上如下所示： 
    obj_t *obj_step = (obj_t*) malloc(sizeof(obj_t));
//... set members of obj_step
newstep_button->obj = *obj_step;   //this will copy the entire struct
    因此您实际上并不需要
    obj_step
    ：您可以直接设置
    newstep_button->obj
    的成员，例如： 
    newstep_button->obj.x = 2;
        
    2020-01-24 0 0
    分享

    室邢

    我看不到ѭ14被设置为任何值；可能没有设置它，或者将其设置为指向未由ѭ15分配的块，而这正是失败的根源。     
    2020-01-24 0 0
    分享

    泪琉踞檄

    您没有将malloc返回的指针存储在button_t对象中。 更改 
    typedef struct
{
    obj_t    obj;     /**< object structure */
    string_t label;   /**< button label, NULL if no label */
    color_t  color;   /**< color of the button */
} button_t;
    至 
    typedef struct
{
    obj_t    *obj;     /**< object structure */
    string_t *label;   /**< button label, NULL if no label */
    color_t  color;   /**< color of the button */
} button_t;
    以及这些init更改： 
    newstep_button->obj = obj_step;
newstep_button->label = label_step;
newstep_button->color = RED;
    这些更改是免费的： 
    button_t *newstep_button = (button_t*) objects[i];
obj_t *obj_step = newstep_button->obj;                
string_t *label_step = newstep_button->label;
free(label_step->text);
free(label_step);
free(obj_step);
free(newstep_button);
    如果释放的内容不是使用malloc创建的，则您所描述的听起来像是典型行为。您基本上是在浪费内存分配列表-当malloc试图找到一些可用内存时，它就会崩溃。 我将不得不看更多的代码来确定，但是您的所有免费声明在我看来都是可疑的。为了找出是否存在问题，请显示代码，您在其中为以下每个语句分配空间： 
    free(label_step->text);
free(label_step);
free(obj_step);
free(newstep_button);
        
    2020-01-24 0 0
    分享
    为什么被折叠? 0 个回复被折叠

    要回复问题请先登录或注册

    或代码 OrCode.com 备案号：粤ICP备15020848号-1
    Escape time: 0.038928031921387