Re: [PATCH] scsi: sr: add handling of memory allocation failure in get_capabilities

From: Martin K. Petersen
Date: Tue Apr 26 2022 - 08:47:38 EST



Enze,

> @@ -669,8 +669,9 @@ static int sr_probe(struct device *dev)
>
> sdev->sector_size = 2048; /* A guess, just in case */
>
> - /* FIXME: need to handle a get_capabilities failure properly ?? */
> - get_capabilities(cd);
> + error = -ENOMEM;
> + if (get_capabilities(cd))
> + goto fail_put;

Shouldn't this be 'goto fail_minor;'?

--
Martin K. Petersen Oracle Linux Engineering