On Wed, Apr 05 2023, Greg Kroah-Hartman wrote: > On Wed, Apr 05, 2023 at 04:34:00PM +0200, Pratyush Yadav wrote: >> On Wed, Apr 05 2023, Greg Kroah-Hartman wrote: >> >> > On Wed, Apr 05, 2023 at 03:57:09PM +0200, Pratyush Yadav wrote: >> >> From: Steve French <stfrench@xxxxxxxxxxxxx> >> >> >> >> [ Upstream commit 87f93d82e0952da18af4d978e7d887b4c5326c0b ] >> >> >> >> Add check for null cifs_sb to create_options helper >> >> >> >> Signed-off-by: Steve French <stfrench@xxxxxxxxxxxxx> >> >> Reviewed-by: Amir Goldstein <amir73il@xxxxxxxxx> >> >> Reviewed-by: Aurelien Aptel <aaptel@xxxxxxxx> >> >> Signed-off-by: Pratyush Yadav <ptyadav@xxxxxxxxx> >> >> --- >> >> >> >> Only compile-tested. This was discovered by our static code analysis >> >> tool. I do not use CIFS and do not know how to actually reproduce the >> >> NULL dereference. >> >> >> >> Follow up from [0]. Original patch is at [1]. >> >> >> >> Mandatory text due to licensing terms: >> >> >> >> This bug was discovered and resolved using Coverity Static Analysis >> >> Security Testing (SAST) by Synopsys, Inc. >> > >> > What? That's funny. And nothing I'm going to be adding to the >> > changelog text, sorry, as that's not what is upstream. >> >> That is fine by me. I placed this text below the 3 dashed lines so it >> does _not_ end up in the commit message, but still discloses this >> information. >> >> > Please go poke your lawyers, that's not ok. >> >> Yes, perhaps I should. But let's go forward with this patch since it >> keeps the original commit message? > > It's already been queued up, you should have gotten an email saying > that, right? Yes, I did, thanks! There was a bit of a race in me sending that email and receiving the notification. -- Regards, Pratyush Yadav Amazon Development Center Germany GmbH Krausenstr. 38 10117 Berlin Geschaeftsfuehrung: Christian Schlaeger, Jonathan Weiss Eingetragen am Amtsgericht Charlottenburg unter HRB 149173 B Sitz: Berlin Ust-ID: DE 289 237 879
