RepoMirrors
/
kpatch
mirror of https://github.com/dynup/kpatch
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

create-diff-object: cleanup maybe-uninitialized compiler complaints 

User disaster123 reports the following build errors:

  create-diff-object.c: In function 'kpatch_process_special_sections':
  create-diff-object.c:2215:41: error: 'key' may be used uninitialized in this function [-Werror=maybe-uninitialized]
         code->sym->name, code->addend, key->sym->name);
                                           ^~
  create-diff-object.c:2138:22: note: 'key' was declared here
    struct rela *code, *key, *rela;
                        ^~~
  In file included from kpatch-elf.h:26,
                   from create-diff-object.c:53:
  log.h:20:3: error: 'code' may be used uninitialized in this function [-Werror=maybe-uninitialized]
     printf(format, ##__VA_ARGS__); \
     ^~~~~~
  create-diff-object.c:2138:15: note: 'code' was declared here
    struct rela *code, *key, *rela;
                 ^~~~
  cc1: all warnings being treated as errors

These are reproducible when building with 9.3.1 and 8.3.1 when building
with optimization level > 2 ( CFLAGS=-O2 make ).  Fix them by
initializing the reported variables to NULL and verifying that they are
infact non-NULL after processing the __jump_table.

Signed-off-by: Joe Lawrence <joe.lawrence@redhat.com>
This commit is contained in:
Joe Lawrence 2020-09-09 13:49:43 -04:00
parent 8cc7ebf488
commit 25f12681fa
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
kpatch-build/create-diff-object.c
View File

@ -2135,7 +2135,7 @@ static bool should_keep_jump_label(struct lookup_table *lookup,
unsigned int group_size, unsigned int group_size,
int *jump_labels_found) int *jump_labels_found)
{ {
struct rela *code, *key, *rela; struct rela *code = NULL, *key = NULL, *rela;
bool tracepoint = false, dynamic_debug = false; bool tracepoint = false, dynamic_debug = false;
struct lookup_result symbol; struct lookup_result symbol;
int i = 0; int i = 0;
@ -2156,7 +2156,7 @@ static bool should_keep_jump_label(struct lookup_table *lookup,
} }
} }
if (i != 3) if (i != 3 || !key || !code)
ERROR("BUG: __jump_table has an unexpected format"); ERROR("BUG: __jump_table has an unexpected format");
if (!strncmp(key->sym->name, "__tracepoint_", 13)) if (!strncmp(key->sym->name, "__tracepoint_", 13))