-----Original Message----- > From: Antoine Tenart <antoine.ten...@bootlin.com> > Sent: Tuesday, July 30, 2019 10:12 AM > To: Pascal van Leeuwen <pascalv...@gmail.com> > Cc: linux-crypto@vger.kernel.org; antoine.ten...@bootlin.com; > herb...@gondor.apana.org.au; > da...@davemloft.net; Pascal Van Leeuwen <pvanleeu...@verimatrix.com> > Subject: Re: [PATCH] crypto: inside-secure - Fix null ptr derefence on rmmod > for > macchiatobin > > Hi Pascal, > > On Fri, Jul 26, 2019 at 05:31:09PM +0200, Pascal van Leeuwen wrote: > > This small patch fixes a null pointer derefence panic that occurred when > > unloading the driver (using rmmod) on macchiatobin due to not setting > > the platform driver data properly in the probe routine. > > > > Signed-off-by: Pascal van Leeuwen <pvanleeu...@verimatrix.com> > > As this is a fix you should add a Fixes: tag so that the patch gets > applied to stable trees. You can have a look at what this tag looks like > at: https://www.kernel.org/doc/html/latest/process/submitting-patches.html > As you already figured out by now, this patch just fixes something that was broken by one of my earlier patches (which has not been applied just yet). So I don't think it applies to stable trees.
> > --- > > drivers/crypto/inside-secure/safexcel.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/crypto/inside-secure/safexcel.c > > b/drivers/crypto/inside- > secure/safexcel.c > > index 45443bf..423ea2d 100644 > > --- a/drivers/crypto/inside-secure/safexcel.c > > +++ b/drivers/crypto/inside-secure/safexcel.c > > @@ -1274,6 +1274,8 @@ static int safexcel_probe(struct platform_device > > *pdev) > > priv->dev = dev; > > priv->version = (enum > > safexcel_eip_version)of_device_get_match_data(dev); > > > > + platform_set_drvdata(pdev, priv); > > + > > This is already done in safexcel_probe(), near the end of the function. > I think you should remove the second call, to avoid setting the platform > driver data twice. > Well, actually, my first patch that you are reviewing right now accidentally removed that other call to platform_set_drvdata ... > Out of curiosity, why calling platform_set_drvdata() earlier in the > probe fixes unloading the driver with rmmod? > Not really. As long as it does get set somewhere :-) > Thanks! > Antoine > > -- > Antoine Ténart, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com Regards, Pascal van Leeuwen Silicon IP Architect, Multi-Protocol Engines @ Verimatrix www.insidesecure.com