At Thu, 24 May 2012 19:32:29 +0530, Devendra Naga wrote:

Signed-off-by: Devendra Naga devendra.aaru@gmail.com

I don't think removing it improves anything much. It's cast to __user pointer, and this rather helps to catch the caution.

thanks,

Takashi

---

sound/pci/emu10k1/emufx.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sound/pci/emu10k1/emufx.c b/sound/pci/emu10k1/emufx.c index dae4050..272f8ac 100644 --- a/sound/pci/emu10k1/emufx.c +++ b/sound/pci/emu10k1/emufx.c @@ -1815,7 +1815,7 @@ static int __devinit _snd_emu10k1_init_efx(struct snd_emu10k1 *emu)

if ((icode = kzalloc(sizeof(*icode), GFP_KERNEL)) == NULL) return -ENOMEM;

• if ((icode->gpr_map = (u_int32_t __user *)

• if ((icode->gpr_map = kcalloc(256 + 160 + 160 + 2 * 512, sizeof(u_int32_t), GFP_KERNEL)) == NULL || (controls = kcalloc(SND_EMU10K1_GPR_CONTROLS,

-- 1.7.9.5