Merge pull request #1143 from joe-lawrence/create-diff-object-uninit

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

Author: joe-lawrence

.travis.yml

@@ -3,6 +3,14 @@ before_install:
   - sudo apt-get -qq update
   - sudo apt-get install -y libelf-dev linux-headers-$(uname -r) shellcheck elfutils
 
+jobs:
+  include:
+    - name: "Default"
+    - name: "-O2"
+      env: CFLAGS="-O2"
+    - name: "-O3"
+      env: CFLAGS="-O3"
+
 script:
   - make
   - make unit

kpatch-build/create-diff-object.c

@@ -2135,7 +2135,7 @@ static bool should_keep_jump_label(struct lookup_table *lookup,
 				   unsigned int group_size,
 				   int *jump_labels_found)
 {
-	struct rela *code, *key, *rela;
+	struct rela *code = NULL, *key = NULL, *rela;
 	bool tracepoint = false, dynamic_debug = false;
 	struct lookup_result symbol;
 	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");
 
 	if (!strncmp(key->sym->name, "__tracepoint_", 13))