Skip to content

Commit

Permalink
config_map: fix configuration type cast issue found on s390x (#8904)
Browse files Browse the repository at this point in the history
Signed-off-by: YingJie Fu <[email protected]>
  • Loading branch information
rightblank authored Aug 19, 2024
1 parent 1a19703 commit fe87949
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions src/flb_config_map.c
Original file line number Diff line number Diff line change
Expand Up @@ -583,7 +583,7 @@ int flb_config_map_set(struct mk_list *properties, struct mk_list *map, void *co
int ret;
int len;
char *base;
char *m_bool;
int *m_bool;
int *m_i_num;
double *m_d_num;
size_t *m_s_num;
Expand Down Expand Up @@ -651,10 +651,10 @@ int flb_config_map_set(struct mk_list *properties, struct mk_list *map, void *co
}
else if (m->type == FLB_CONFIG_MAP_TIME) {
m_i_num = (int *) (base + m->offset);
*m_i_num = m->value.val.s_num;
*m_i_num = m->value.val.i_num;
}
else if (m->type == FLB_CONFIG_MAP_BOOL) {
m_bool = (char *) (base + m->offset);
m_bool = (int *) (base + m->offset);
*m_bool = m->value.val.boolean;
}
else if (m->type >= FLB_CONFIG_MAP_CLIST ||
Expand Down Expand Up @@ -779,7 +779,7 @@ int flb_config_map_set(struct mk_list *properties, struct mk_list *map, void *co
*m_d_num = atof(kv->val);
}
else if (m->type == FLB_CONFIG_MAP_BOOL) {
m_bool = (char *) (base + m->offset);
m_bool = (int *) (base + m->offset);
ret = flb_utils_bool(kv->val);
if (ret == -1) {
flb_error("[config map] invalid value for boolean property '%s=%s'",
Expand Down
2 changes: 1 addition & 1 deletion tests/internal/config_map.c
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ struct context {
/* Single values */
int num_int;
size_t size;
time_t time;
int time;
int boolean;
double num_double;
flb_sds_t string;
Expand Down

0 comments on commit fe87949

Please sign in to comment.