arch: tile: kernel: unaligned.c: Cleaning up uninitialized variables
authorRickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Sat, 31 May 2014 23:00:13 +0000 (01:00 +0200)
committerChris Metcalf <cmetcalf@tilera.com>
Sun, 8 Jun 2014 01:39:11 +0000 (21:39 -0400)
There is a risk that the variable will be used without being initialized.

This was largely found by using a static code analysis program called cppcheck.

Signed-off-by: Rickard Strandqvist <rickard_strandqvist@spectrumdigital.se>
Signed-off-by: Chris Metcalf <cmetcalf@tilera.com> [minor cleanups]
arch/tile/kernel/unaligned.c

index b030b4e78845b596c1fc429687e360a2791e7046..c02ea2a45f679c056a9162911a0c0f0128ace0f9 100644 (file)
@@ -182,18 +182,7 @@ static void find_regs(tilegx_bundle_bits bundle, uint64_t *rd, uint64_t *ra,
        int i;
        uint64_t reg;
        uint64_t reg_map = 0, alias_reg_map = 0, map;
-       bool alias;
-
-       *ra = -1;
-       *rb = -1;
-
-       if (rd)
-               *rd = -1;
-
-       *clob1 = -1;
-       *clob2 = -1;
-       *clob3 = -1;
-       alias = false;
+       bool alias = false;
 
        /*
         * Parse fault bundle, find potential used registers and mark
@@ -569,7 +558,7 @@ void jit_bundle_gen(struct pt_regs *regs, tilegx_bundle_bits bundle,
        tilegx_bundle_bits bundle_2 = 0;
        /* If bundle_2_enable = false, bundle_2 is fnop/nop operation. */
        bool     bundle_2_enable = true;
-       uint64_t ra, rb, rd = -1, clob1, clob2, clob3;
+       uint64_t ra = -1, rb = -1, rd = -1, clob1 = -1, clob2 = -1, clob3 = -1;
        /*
         * Indicate if the unalign access
         * instruction's registers hit with