scsi: libsas: Fix an error code in sas_ata_add_dev()
authorDan Carpenter <error27@gmail.com>
Thu, 5 Jan 2023 10:53:35 +0000 (13:53 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 12 Jan 2023 04:32:53 +0000 (23:32 -0500)
This code accidentally returns success instead of -ENOMEM.

Fixes: 7cc7646b4b24 ("scsi: libsas: Factor out sas_ata_add_dev()")
Link: https://lore.kernel.org/r/Y7asLxzVwQ56G+ya@kili
Signed-off-by: Dan Carpenter <error27@gmail.com>
Reviewed-by: Jason Yan <yanaijie@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/libsas/sas_ata.c

index 9b4a9c3..4a7e835 100644 (file)
@@ -716,7 +716,7 @@ int sas_ata_add_dev(struct domain_device *parent, struct ex_phy *phy,
 
        rphy = sas_end_device_alloc(phy->port);
        if (!rphy)
-               return ret;
+               return -ENOMEM;
 
        rphy->identify.phy_identifier = phy_id;
        child->rphy = rphy;