aboutsummaryrefslogtreecommitdiffstats
path: root/wiretap/merge.c
diff options
context:
space:
mode:
authorGuy Harris <gharris@sonic.net>2022-03-14 12:11:24 -0700
committerGuy Harris <gharris@sonic.net>2022-03-14 19:12:20 +0000
commit109b92b5d796f2fb64491fe1976257bc976f1f40 (patch)
treefa9fe0a1d1e9dc33aa6cb25774bf19ae3a53d60b /wiretap/merge.c
parent304dae33d68d923cd2da6d405d999f8a8527ba71 (diff)
wiretap: have wtap_dump_close() provide a "needs to be reloaded" indication.
This allows the "needs to be reloaded" indication to be set in the close process, as is the case for ERF; having a routine that returns the value of that indication is not useful if it gets seet in the close process, as the handle for the wtap_dumper is no longer valid after wtap_dump_close() finishes. We also get rid of wtap_dump_get_needs_reload(), as callers should get that information via the added argument to wtap_dump_close(). Fixes #17989.
Diffstat (limited to 'wiretap/merge.c')
-rw-r--r--wiretap/merge.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/wiretap/merge.c b/wiretap/merge.c
index da4e20b585..05a48ad90a 100644
--- a/wiretap/merge.c
+++ b/wiretap/merge.c
@@ -952,7 +952,7 @@ merge_process_packets(wtap_dumper *pdh, const int file_type,
cb->callback_func(MERGE_EVENT_DONE, count, in_files, in_file_count, cb->data);
if (status == MERGE_OK || status == MERGE_USER_ABORTED) {
- if (!wtap_dump_close(pdh, err, err_info))
+ if (!wtap_dump_close(pdh, NULL, err, err_info))
status = MERGE_ERR_CANT_CLOSE_OUTFILE;
} else {
/*
@@ -963,7 +963,7 @@ merge_process_packets(wtap_dumper *pdh, const int file_type,
*/
int close_err = 0;
gchar *close_err_info = NULL;
- (void)wtap_dump_close(pdh, &close_err, &close_err_info);
+ (void)wtap_dump_close(pdh, NULL, &close_err, &close_err_info);
g_free(close_err_info);
}