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
