diff options
author | Guy Harris <guy@alum.mit.edu> | 2016-02-10 17:48:46 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2016-02-11 01:49:49 +0000 |
commit | 01a3873505e6f5c6b0b92914345478ff250d5248 (patch) | |
tree | d4ead072c844a32a4c6ed6f721f47c484c3b5748 /wiretap/merge.c | |
parent | 5cffcf20b147ac88d58774ccce0b76db4704b329 (diff) | |
download | wireshark-01a3873505e6f5c6b0b92914345478ff250d5248.tar.gz |
Make file indices consistently unsigned in the merge code.
Change-Id: Ie531e5c24c32ca86abbe0c2b5d3526014e5fb831
Reviewed-on: https://code.wireshark.org/review/13897
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap/merge.c')
-rw-r--r-- | wiretap/merge.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/wiretap/merge.c b/wiretap/merge.c index 06dc5db617..b4e1438b0e 100644 --- a/wiretap/merge.c +++ b/wiretap/merge.c @@ -119,12 +119,12 @@ add_idb_index_map(merge_in_file_t *in_file, const guint orig_index, const guint * @return TRUE if all files could be opened, FALSE otherwise */ static gboolean -merge_open_in_files(int in_file_count, const char *const *in_file_names, +merge_open_in_files(guint in_file_count, const char *const *in_file_names, merge_in_file_t **in_files, int *err, gchar **err_info, - int *err_fileno) + guint *err_fileno) { - gint i; - gint j; + guint i; + guint j; size_t files_size = in_file_count * sizeof(merge_in_file_t); merge_in_file_t *files; gint64 size; @@ -148,7 +148,7 @@ merge_open_in_files(int in_file_count, const char *const *in_file_names, } size = wtap_file_size(files[i].wth, err); if (size == -1) { - for (j = 0; j != G_MAXINT && j <= i; j++) + for (j = 0; j != G_MAXUINT && j <= i; j++) cleanup_in_file(&files[j]); *err_fileno = i; return FALSE; @@ -842,7 +842,7 @@ merge_files(int out_fd, const gchar* out_filename, const int file_type, const char *const *in_filenames, const guint in_file_count, const gboolean do_append, const idb_merge_mode mode, guint snaplen, const gchar *app_name, merge_progress_callback_t* cb, - int *err, gchar **err_info, int *err_fileno) + int *err, gchar **err_info, guint *err_fileno) { merge_in_file_t *in_files = NULL, *in_file = NULL; int frame_type = WTAP_ENCAP_PER_PACKET; |