diff options
author | Guy Harris <guy@alum.mit.edu> | 2015-11-09 11:54:18 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2015-11-09 19:55:13 +0000 |
commit | 09f5ff4fc6895f375b3ea07f6151d8e09377fd52 (patch) | |
tree | 5829d926242a6297803d52417c41ea5512536c23 /wiretap/file_access.c | |
parent | fe12759b8e0370dd3e4e8434b4140c098ade7a38 (diff) |
Call the dumper routine to finish write a file the "finish" routine.
It doesn't actually *close* any handle, so it's best called a "finish"
routine rather than a "close" routine.
In libwiretap modules, don't bother setting the finish routine pointer
to null - it's already initialized to null (it's probably best not to
require modules to set it).
Change-Id: I19554f3fb826db495f17b36600ae36222cbc21b0
Reviewed-on: https://code.wireshark.org/review/11659
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap/file_access.c')
-rw-r--r-- | wiretap/file_access.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/wiretap/file_access.c b/wiretap/file_access.c index 9d59f1274c..15e5940ac7 100644 --- a/wiretap/file_access.c +++ b/wiretap/file_access.c @@ -2436,9 +2436,9 @@ wtap_dump_close(wtap_dumper *wdh, int *err) { gboolean ret = TRUE; - if (wdh->subtype_close != NULL) { - /* There's a close routine for this dump stream. */ - if (!(wdh->subtype_close)(wdh, err)) + if (wdh->subtype_finish != NULL) { + /* There's a finish routine for this dump stream. */ + if (!(wdh->subtype_finish)(wdh, err)) ret = FALSE; } errno = WTAP_ERR_CANT_CLOSE; |