lkml.org 
[lkml]   [2018]   [Dec]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[GIT PULL 02/21] lightnvm: Fix uninitialized return value in nvm_get_chunk_meta()
    Date
    From: Geert Uytterhoeven <geert@linux-m68k.org>

    With gcc 4.1:

    drivers/lightnvm/core.c: In function ‘nvm_get_bb_meta’:
    drivers/lightnvm/core.c:977: warning: ‘ret’ may be used uninitialized in this function

    and

    drivers/nvme/host/lightnvm.c: In function ‘nvme_nvm_get_chk_meta’:
    drivers/nvme/host/lightnvm.c:580: warning: ‘ret’ may be used uninitialized in this function

    Indeed, if (for the former) the number of channels or LUNs is zero, or
    (for both) the passed number of chunks is zero, ret will be returned
    uninitialized.

    Fix this by preinitializing ret to zero.

    Fixes: aff3fb18f957de93 ("lightnvm: move bad block and chunk state logic to core")
    Fixes: a294c199455187d1 ("lightnvm: implement get log report chunk helpers")
    Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org>
    Signed-off-by: Matias Bjørling <mb@lightnvm.io>
    ---
    drivers/lightnvm/core.c | 2 +-
    drivers/nvme/host/lightnvm.c | 3 ++-
    2 files changed, 3 insertions(+), 2 deletions(-)

    diff --git a/drivers/lightnvm/core.c b/drivers/lightnvm/core.c
    index 60ab11fcc81c..10e541cb8dc3 100644
    --- a/drivers/lightnvm/core.c
    +++ b/drivers/lightnvm/core.c
    @@ -974,7 +974,7 @@ static int nvm_get_bb_meta(struct nvm_dev *dev, sector_t slba,
    struct ppa_addr ppa;
    u8 *blks;
    int ch, lun, nr_blks;
    - int ret;
    + int ret = 0;

    ppa.ppa = slba;
    ppa = dev_to_generic_addr(dev, ppa);
    diff --git a/drivers/nvme/host/lightnvm.c b/drivers/nvme/host/lightnvm.c
    index a4f3b263cd6c..d64805dc8efb 100644
    --- a/drivers/nvme/host/lightnvm.c
    +++ b/drivers/nvme/host/lightnvm.c
    @@ -577,7 +577,8 @@ static int nvme_nvm_get_chk_meta(struct nvm_dev *ndev,
    struct ppa_addr ppa;
    size_t left = nchks * sizeof(struct nvme_nvm_chk_meta);
    size_t log_pos, offset, len;
    - int ret, i, max_len;
    + int i, max_len;
    + int ret = 0;

    /*
    * limit requests to maximum 256K to avoid issuing arbitrary large
    --
    2.17.1
    \
     
     \ /
      Last update: 2018-12-11 20:19    [W:2.606 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site