From 7feb38651970fc1f3cfd90769685c46804ab25ae Mon Sep 17 00:00:00 2001 From: "Olle E. Johansson" Date: Mon, 3 Jan 2022 16:33:08 +0100 Subject: [PATCH] secfilter: Convert error messages to error responses in secf_rpc_add_dst() --- src/modules/secfilter/secfilter_rpc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/modules/secfilter/secfilter_rpc.c b/src/modules/secfilter/secfilter_rpc.c index cea9d400e83..2f763e4732d 100644 --- a/src/modules/secfilter/secfilter_rpc.c +++ b/src/modules/secfilter/secfilter_rpc.c @@ -63,7 +63,7 @@ void secf_rpc_add_dst(rpc_t *rpc, void *ctx) char *text = NULL; if(rpc->scan(ctx, "d", &number) < 1) { - rpc->fault(ctx, 0, + rpc->fault(ctx, 500, "Invalid Parameters. Usage: secfilter.add_dst " "number\n Example: secfilter.add_dst " "555123123"); @@ -73,7 +73,7 @@ void secf_rpc_add_dst(rpc_t *rpc, void *ctx) data.s = pkg_malloc(data.len * sizeof(char)); if(!data.s) { PKG_MEM_ERROR; - rpc->rpl_printf(ctx, "Error insert values in the blacklist"); + rpc->fault(ctx, 500, "Error insert values in the blacklist"); return; } memcpy(data.s, text, data.len); @@ -82,7 +82,7 @@ void secf_rpc_add_dst(rpc_t *rpc, void *ctx) rpc->rpl_printf(ctx, "Values (%s) inserted into blacklist destinations", data); } else { - rpc->rpl_printf(ctx, "Error insert values in the blacklist"); + rpc->fault(ctx, 500, "Error insert values in the blacklist"); } lock_release(&secf_data->lock); if(data.s)