From 118dfc08b89ba185968664d65a82482b884710ce Mon Sep 17 00:00:00 2001 From: ZhouBox Date: Thu, 9 Apr 2026 10:10:58 +0800 Subject: [PATCH] fix(core): correct parameter order in neu_req_del_tag_copy - Fix reversed parameter order in neu_req_del_tag_copy call at manager.c:1488 - Add missing return statement on strdup failure in neu_req_del_tag_copy function --- include/neuron/msg.h | 1 + src/core/manager.c | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/include/neuron/msg.h b/include/neuron/msg.h index 6d68d9307..99442fada 100644 --- a/include/neuron/msg.h +++ b/include/neuron/msg.h @@ -721,6 +721,7 @@ static inline int neu_req_del_tag_copy(neu_req_del_tag_t *dst, free(dst->tags[i]); } free(dst->tags); + return -1; } } return 0; diff --git a/src/core/manager.c b/src/core/manager.c index 4a974bd79..df8ffe1c3 100644 --- a/src/core/manager.c +++ b/src/core/manager.c @@ -1485,7 +1485,7 @@ static int manager_loop(enum neu_event_io_type type, int fd, void *usr_data) neu_msg_get_header(msg_copy); neu_req_del_tag_t *cmd_copy = (neu_req_del_tag_t *) &header_copy[1]; - neu_req_del_tag_copy(cmd, cmd_copy); + neu_req_del_tag_copy(cmd_copy, cmd); forward_msg(manager, header_copy, app->app_name); }