NULL pointer in imx-sdma.c with from context arg in reject_firmware_nowait
Denis 'GNUtoo' Carikli
GNUtoo at no-log.org
Mon May 28 16:31:53 UTC 2018
On Tue, 17 Apr 2018 10:41:29 +0200
Xavi Drudis Ferran <xdrudis at tinet.cat> wrote:
The patch below seem sensible to me:
> request_firmware_nowait(module, uevent, NONFREE_FIRMWARE,
> - device, gfp, NULL, cont);
> + device, gfp, context, cont);
However merging the patch without finding why the NULL was put there
doesn't seem a good idea as it could break many drivers along the way.
Would it be possible to try to find this information by:
- Finding when it was first introduced
- By looking at the ChangeLogs of the version introducing it
Note that I'm not a Linux-libre maintainer, here I'm just helping to
review the patch.
Denis.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://www.fsfla.org/pipermail/linux-libre/attachments/20180528/94179348/attachment.sig>
More information about the linux-libre
mailing list