| From 399f2c9a3acd5bd913e50a4dde52dee6527b297e Mon Sep 17 00:00:00 2001 |
| From: Kairui Song <kasong@redhat.com> |
| Date: Wed, 29 Jan 2020 13:37:13 +0800 |
| Subject: [PATCH] Remove duplicated variable definitions |
| |
| When building on Fedora 32 (with GCC 10), following error is observed: |
| |
| /usr/bin/ld: erase_info.o:/tmp/makedumpfile/makedumpfile.h:2010: multiple definition of |
| `crash_reserved_mem_nr'; elf_info.o:/tmp/makedumpfile/makedumpfile.h:2010: first defined here |
| /usr/bin/ld: erase_info.o:/tmp/makedumpfile/makedumpfile.h:2009: multiple definition of |
| `crash_reserved_mem'; elf_info.o:/tmp/makedumpfile/makedumpfile.h:2009: first defined here |
| /usr/bin/ld: erase_info.o:/tmp/makedumpfile/makedumpfile.h:1278: multiple definition of |
| `parallel_info_t'; elf_info.o:/tmp/makedumpfile/makedumpfile.h:1278: first defined here |
| /usr/bin/ld: erase_info.o:/tmp/makedumpfile/makedumpfile.h:1265: multiple definition of |
| `splitting_info_t'; elf_info.o:/tmp/makedumpfile/makedumpfile.h:1265: first defined here |
| ... |
| collect2: error: ld returned 1 exit status |
| make: *** [Makefile:97: makedumpfile] Error 1 |
| |
| These variables are wrongly defined multiple times. So remove the |
| duplicated definitions. |
| |
| Signed-off-by: Kairui Song <kasong@redhat.com> |
| Signed-off-by: Kazuhito Hagio <k-hagio-ab@nec.com> |
| Signed-off-by: Alexander Egorenkov <egorenar-dev@posteo.net> |
| --- |
| makedumpfile.c | 8 ++++---- |
| makedumpfile.h | 8 ++++---- |
| 2 files changed, 8 insertions(+), 8 deletions(-) |
| |
| diff --git a/makedumpfile.c b/makedumpfile.c |
| index e290fbd..ae7336a 100644 |
| --- a/makedumpfile.c |
| +++ b/makedumpfile.c |
| @@ -10954,7 +10954,7 @@ check_param_for_reassembling_dumpfile(int argc, char *argv[]) |
| return FALSE; |
| |
| if ((info->splitting_info |
| - = malloc(sizeof(splitting_info_t) * info->num_dumpfile)) |
| + = malloc(sizeof(struct splitting_info) * info->num_dumpfile)) |
| == NULL) { |
| MSG("Can't allocate memory for splitting_info.\n"); |
| return FALSE; |
| @@ -11042,7 +11042,7 @@ check_param_for_creating_dumpfile(int argc, char *argv[]) |
| return FALSE; |
| } |
| if ((info->splitting_info |
| - = malloc(sizeof(splitting_info_t) * info->num_dumpfile)) |
| + = malloc(sizeof(struct splitting_info) * info->num_dumpfile)) |
| == NULL) { |
| MSG("Can't allocate memory for splitting_info.\n"); |
| return FALSE; |
| @@ -11077,13 +11077,13 @@ check_param_for_creating_dumpfile(int argc, char *argv[]) |
| |
| if (info->num_threads) { |
| if ((info->parallel_info = |
| - malloc(sizeof(parallel_info_t) * info->num_threads)) |
| + malloc(sizeof(struct parallel_info) * info->num_threads)) |
| == NULL) { |
| MSG("Can't allocate memory for parallel_info.\n"); |
| return FALSE; |
| } |
| |
| - memset(info->parallel_info, 0, sizeof(parallel_info_t) |
| + memset(info->parallel_info, 0, sizeof(struct parallel_info) |
| * info->num_threads); |
| } |
| |
| diff --git a/makedumpfile.h b/makedumpfile.h |
| index 68d9691..7217407 100644 |
| --- a/makedumpfile.h |
| +++ b/makedumpfile.h |
| @@ -1262,7 +1262,7 @@ struct splitting_info { |
| mdf_pfn_t end_pfn; |
| off_t offset_eraseinfo; |
| unsigned long size_eraseinfo; |
| -} splitting_info_t; |
| +}; |
| |
| struct parallel_info { |
| int fd_memory; |
| @@ -1275,7 +1275,7 @@ struct parallel_info { |
| #ifdef USELZO |
| lzo_bytep wrkmem; |
| #endif |
| -} parallel_info_t; |
| +}; |
| |
| struct ppc64_vmemmap { |
| unsigned long phys; |
| @@ -2006,8 +2006,8 @@ struct memory_range { |
| }; |
| |
| #define CRASH_RESERVED_MEM_NR 8 |
| -struct memory_range crash_reserved_mem[CRASH_RESERVED_MEM_NR]; |
| -int crash_reserved_mem_nr; |
| +extern struct memory_range crash_reserved_mem[CRASH_RESERVED_MEM_NR]; |
| +extern int crash_reserved_mem_nr; |
| |
| unsigned long read_vmcoreinfo_symbol(char *str_symbol); |
| int readmem(int type_addr, unsigned long long addr, void *bufptr, size_t size); |
| -- |
| 2.28.0 |
| |