Catégorie : Linux

firmware: fix possible use after free on name on asynchronous request

Author: Luis R. Rodriguez <mcgrof@suse.com> Asynchronous firmware loading copies the pointer to the name passed as an argument only to be scheduled later and used. This behaviour works well for synchronous calling but in asynchronous mode there’s a chance the caller could immediately free the passed string after making the asynchronous call. This could trigger …

Lire la suite

carl9170: match wait_for_completion_timeout return type

Author: Nicholas Mc Guire <hofrat@osadl.org> Return type of wait_for_completion_timeout is unsigned long not int. An appropriately named unsigned long is added, and the assignments as well as error checking fixed up. API conformance testing for completions with coccinelle spatches are being used to locate API usage inconsistencies: ./drivers/net/wireless/ath/carl9170/usb.c:675 int return assigned to unsigned long Patch …

Lire la suite

ASoC: max98095: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40eb76de8adf039409722015927dc3 (« genirq: Reject bogus threaded irq requests ») threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Mark Brown …

Lire la suite

ASoC: twl6040: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40eb76de8adf039409722015927dc3 (« genirq: Reject bogus threaded irq requests ») threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Mark Brown …

Lire la suite

ASoC: wm8994: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40eb76de8adf039409722015927dc3 (« genirq: Reject bogus threaded irq requests ») threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Mark Brown …

Lire la suite

ASoC: wm5100: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40eb76de8adf039409722015927dc3 (« genirq: Reject bogus threaded irq requests ») threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Mark Brown …

Lire la suite

ASoC: wm8996: Pass the IRQF_ONESHOT flag

Author: Fabio Estevam <fabio.estevam@freescale.com> Since commit 1c6c69525b40eb76de8adf039409722015927dc3 (« genirq: Reject bogus threaded irq requests ») threaded IRQs without a primary handler need to be requested with IRQF_ONESHOT, otherwise the request will fail. So pass the IRQF_ONESHOT flag in this case. The semantic patch that makes this change is available in scripts/coccinelle/misc/irqf_oneshot.cocci. Signed-off-by: Fabio Estevam Signed-off-by: Mark Brown …

Lire la suite

rcu: Change function declaration to bool

Author: Nicholas Mc Guire <hofrat@osadl.org> rcu_cpu_has_callbacks() is declared int. The current declaration was introduced in commit c0f4dfd4f90f (rcu: Make RCU_FAST_NO_HZ take advantage of numbered callbacks). But it is actually returning bool and as the function description states  » * Return true if the specified CPU has any callback…. », this probably should be a bool as …

Lire la suite

mmc: host: sdhci: Use BUG_ON()

Author: Fabio Estevam <fabio.estevam@freescale.com> Use BUG_ON() instead of an ‘if’ condition followed by BUG(). The semantic patch that makes this change is available in scripts/coccinelle/misc/bugon.cocci. More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Fabio Estevam Signed-off-by: Ulf Hansson — drivers/mmc/host/sdhci.c | 6 ++—- 1 file changed, 2 insertions(+), 4 deletions(-)   diff –git …

Lire la suite

mmc: host: mxcmmc: Simplify a trivial if-return sequence

Author: Fabio Estevam <fabio.estevam@freescale.com> Simplify a trivial if-return sequence. Possibly combine with a preceding function call. The semantic patch that makes this change is available in scripts/coccinelle/misc/simple_return.cocci. More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: Fabio Estevam Signed-off-by: Ulf Hansson — drivers/mmc/host/mxcmmc.c | 6 +—– 1 file changed, 1 insertion(+), 5 deletions(-)   …

Lire la suite