lkml.org 
[lkml]   [2020]   [Sep]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.8 053/118] scsi: libsas: Fix error path in sas_notify_lldd_dev_found()
    Date
    From: Dan Carpenter <dan.carpenter@oracle.com>

    [ Upstream commit 244359c99fd90f1c61c3944f93250f8219435c75 ]

    In sas_notify_lldd_dev_found(), if we can't allocate the necessary
    resources, then it seems like the wrong thing to mark the device as found
    and to increment the reference count. None of the callers ever drop the
    reference in that situation.

    [mkp: tweaked commit desc based on feedback from John]

    Link: https://lore.kernel.org/r/20200905125836.GF183976@mwanda
    Fixes: 735f7d2fedf5 ("[SCSI] libsas: fix domain_device leak")
    Reviewed-by: Jason Yan <yanaijie@huawei.com>
    Acked-by: John Garry <john.garry@huawei.com>
    Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
    Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/scsi/libsas/sas_discover.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    diff --git a/drivers/scsi/libsas/sas_discover.c b/drivers/scsi/libsas/sas_discover.c
    index daf951b0b3f55..13ad2b3d314e2 100644
    --- a/drivers/scsi/libsas/sas_discover.c
    +++ b/drivers/scsi/libsas/sas_discover.c
    @@ -182,10 +182,11 @@ int sas_notify_lldd_dev_found(struct domain_device *dev)
    pr_warn("driver on host %s cannot handle device %016llx, error:%d\n",
    dev_name(sas_ha->dev),
    SAS_ADDR(dev->sas_addr), res);
    + return res;
    }
    set_bit(SAS_DEV_FOUND, &dev->state);
    kref_get(&dev->kref);
    - return res;
    + return 0;
    }


    --
    2.25.1


    \
     
     \ /
      Last update: 2020-09-21 18:45    [W:5.219 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site