Skip to content

Commit

Permalink
output-file: remove ThreadExitPrintStats callback
Browse files Browse the repository at this point in the history
Not used.

Ticket: OISF#7227
  • Loading branch information
jasonish authored and Corey Thomas committed Sep 3, 2024
1 parent 570ceb9 commit 1ab3563
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 25 deletions.
6 changes: 1 addition & 5 deletions src/output-file.c
Original file line number Diff line number Diff line change
Expand Up @@ -49,15 +49,12 @@ typedef struct OutputFileLogger_ {
LoggerId logger_id;
ThreadInitFunc ThreadInit;
ThreadDeinitFunc ThreadDeinit;
ThreadExitPrintStatsFunc ThreadExitPrintStats;
} OutputFileLogger;

static OutputFileLogger *list = NULL;

int OutputRegisterFileLogger(LoggerId id, const char *name, FileLogger LogFunc,
OutputCtx *output_ctx, ThreadInitFunc ThreadInit,
ThreadDeinitFunc ThreadDeinit,
ThreadExitPrintStatsFunc ThreadExitPrintStats)
OutputCtx *output_ctx, ThreadInitFunc ThreadInit, ThreadDeinitFunc ThreadDeinit)
{
OutputFileLogger *op = SCCalloc(1, sizeof(*op));
if (op == NULL)
Expand All @@ -69,7 +66,6 @@ int OutputRegisterFileLogger(LoggerId id, const char *name, FileLogger LogFunc,
op->logger_id = id;
op->ThreadInit = ThreadInit;
op->ThreadDeinit = ThreadDeinit;
op->ThreadExitPrintStats = ThreadExitPrintStats;

if (list == NULL)
list = op;
Expand Down
5 changes: 2 additions & 3 deletions src/output-file.h
Original file line number Diff line number Diff line change
Expand Up @@ -48,9 +48,8 @@ void OutputFileLogFfc(ThreadVars *tv, OutputFileLoggerThreadData *op_thread_data
typedef int (*FileLogger)(ThreadVars *, void *thread_data, const Packet *, const File *, void *tx,
const uint64_t tx_id, uint8_t direction);

int OutputRegisterFileLogger(LoggerId id, const char *name, FileLogger LogFunc,
OutputCtx *, ThreadInitFunc ThreadInit, ThreadDeinitFunc ThreadDeinit,
ThreadExitPrintStatsFunc ThreadExitPrintStats);
int OutputRegisterFileLogger(LoggerId id, const char *name, FileLogger LogFunc, OutputCtx *,
ThreadInitFunc ThreadInit, ThreadDeinitFunc ThreadDeinit);

void OutputFileLoggerRegister(void);

Expand Down
5 changes: 2 additions & 3 deletions src/output-json-file.c
Original file line number Diff line number Diff line change
Expand Up @@ -352,7 +352,6 @@ static OutputInitResult OutputFileLogInitSub(ConfNode *conf, OutputCtx *parent_c
void JsonFileLogRegister (void)
{
/* register as child of eve-log */
OutputRegisterFileSubModule(LOGGER_JSON_FILE, "eve-log", "JsonFileLog",
"eve-log.files", OutputFileLogInitSub, JsonFileLogger,
JsonFileLogThreadInit, JsonFileLogThreadDeinit, NULL);
OutputRegisterFileSubModule(LOGGER_JSON_FILE, "eve-log", "JsonFileLog", "eve-log.files",
OutputFileLogInitSub, JsonFileLogger, JsonFileLogThreadInit, JsonFileLogThreadDeinit);
}
9 changes: 3 additions & 6 deletions src/output.c
Original file line number Diff line number Diff line change
Expand Up @@ -391,11 +391,9 @@ void OutputRegisterTxSubModule(LoggerId id, const char *parent_name, const char
*
* \retval Returns 0 on success, -1 on failure.
*/
void OutputRegisterFileSubModule(LoggerId id, const char *parent_name,
const char *name, const char *conf_name, OutputInitSubFunc InitFunc,
FileLogger FileLogFunc, ThreadInitFunc ThreadInit,
ThreadDeinitFunc ThreadDeinit,
ThreadExitPrintStatsFunc ThreadExitPrintStats)
void OutputRegisterFileSubModule(LoggerId id, const char *parent_name, const char *name,
const char *conf_name, OutputInitSubFunc InitFunc, FileLogger FileLogFunc,
ThreadInitFunc ThreadInit, ThreadDeinitFunc ThreadDeinit)
{
if (unlikely(FileLogFunc == NULL)) {
goto error;
Expand All @@ -414,7 +412,6 @@ void OutputRegisterFileSubModule(LoggerId id, const char *parent_name,
module->FileLogFunc = FileLogFunc;
module->ThreadInit = ThreadInit;
module->ThreadDeinit = ThreadDeinit;
module->ThreadExitPrintStats = ThreadExitPrintStats;
TAILQ_INSERT_TAIL(&output_modules, module, entries);

SCLogDebug("File logger \"%s\" registered.", name);
Expand Down
8 changes: 3 additions & 5 deletions src/output.h
Original file line number Diff line number Diff line change
Expand Up @@ -116,11 +116,9 @@ void OutputRegisterTxSubModuleWithProgress(LoggerId id, const char *parent_name,
int tc_log_progress, int ts_log_progress, ThreadInitFunc ThreadInit,
ThreadDeinitFunc ThreadDeinit);

void OutputRegisterFileSubModule(LoggerId id, const char *parent_name,
const char *name, const char *conf_name,
OutputInitSubFunc InitFunc, FileLogger FileLogFunc,
ThreadInitFunc ThreadInit, ThreadDeinitFunc ThreadDeinit,
ThreadExitPrintStatsFunc ThreadExitPrintStats);
void OutputRegisterFileSubModule(LoggerId id, const char *parent_name, const char *name,
const char *conf_name, OutputInitSubFunc InitFunc, FileLogger FileLogFunc,
ThreadInitFunc ThreadInit, ThreadDeinitFunc ThreadDeinit);

void OutputRegisterFiledataModule(LoggerId id, const char *name, const char *conf_name,
OutputInitFunc InitFunc, FiledataLogger FiledataLogFunc, ThreadInitFunc ThreadInit,
Expand Down
5 changes: 2 additions & 3 deletions src/runmodes.c
Original file line number Diff line number Diff line change
Expand Up @@ -637,9 +637,8 @@ static void SetupOutput(
filedata_logger_count++;
} else if (module->FileLogFunc) {
SCLogDebug("%s is a file logger", module->name);
OutputRegisterFileLogger(module->logger_id, module->name,
module->FileLogFunc, output_ctx, module->ThreadInit,
module->ThreadDeinit, module->ThreadExitPrintStats);
OutputRegisterFileLogger(module->logger_id, module->name, module->FileLogFunc, output_ctx,
module->ThreadInit, module->ThreadDeinit);
file_logger_count++;
} else if (module->StreamingLogFunc) {
SCLogDebug("%s is a streaming logger", module->name);
Expand Down

0 comments on commit 1ab3563

Please sign in to comment.