]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
mcb: Fix an error handling path in 'chameleon_parse_cells()'
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Tue, 29 Aug 2017 12:47:25 +0000 (14:47 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 31 Aug 2017 16:49:41 +0000 (18:49 +0200)
If 'chameleon_get_bar()' fails, we will return 0, which mean success.
We should return the corresponding error code instead.

Remove the useless initialisation of 'ret' which was hiding the issue.
(if 'ret' is not set, gcc generates a warning ("warning: ‘ret’ may be used
uninitialized in this function"))

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/mcb/mcb-parse.c

index ee7fb6ec96bd17e599905c947e06533b81ce9857..7369bda3442fbc10197a00b35553cbac3c365adf 100644 (file)
@@ -182,7 +182,7 @@ int chameleon_parse_cells(struct mcb_bus *bus, phys_addr_t mapbase,
        int num_cells = 0;
        uint32_t dtype;
        int bar_count;
-       int ret = 0;
+       int ret;
        u32 hsize;
 
        hsize = sizeof(struct chameleon_fpga_header);
@@ -210,8 +210,10 @@ int chameleon_parse_cells(struct mcb_bus *bus, phys_addr_t mapbase,
                 header->filename);
 
        bar_count = chameleon_get_bar(&p, mapbase, &cb);
-       if (bar_count < 0)
+       if (bar_count < 0) {
+               ret = bar_count;
                goto free_header;
+       }
 
        for_each_chameleon_cell(dtype, p) {
                switch (dtype) {