Skip to content

Commit

Permalink
logmpx: move log_msg_write_protect() back to multiplexer
Browse files Browse the repository at this point in the history
Making the message write protected is independent of filterx and is
required by all pipeline elements.

Signed-off-by: László Várady <laszlo.varady@anno.io>
Signed-off-by: Hofi <hofione@gmail.com>
  • Loading branch information
MrAnno authored and HofiOne committed Sep 30, 2024
1 parent cb47fe6 commit 38284c6
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 0 additions & 1 deletion lib/filterx/filterx-eval.h
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ filterx_eval_prepare_for_fork(FilterXEvalContext *context, LogMessage **pmsg, co
filterx_eval_sync_message(context, pmsg, path_options);
if (context)
filterx_scope_write_protect(context->scope);
log_msg_write_protect(*pmsg);
}

#endif
1 change: 1 addition & 0 deletions lib/logmpx.c
Original file line number Diff line number Diff line change
Expand Up @@ -94,6 +94,7 @@ log_multiplexer_queue(LogPipe *s, LogMessage *msg, const LogPathOptions *path_op
* data we still need */

filterx_eval_prepare_for_fork(path_options->filterx_context, &msg, path_options);
log_msg_write_protect(msg);
}
for (fallback = 0; (fallback == 0) || (fallback == 1 && self->fallback_exists && !delivered); fallback++)
{
Expand Down

0 comments on commit 38284c6

Please sign in to comment.