Matthias Schwarzott wrote:
Hi Michael!

Looking at your new lgdt3305 driver, I noticed that the error path of lgdt3305_attach, that is also jumped to kzalloc errors, sets state->frontend.demodulator_priv to NULL.

That will oops in case state is NULL. So you either need two goto labels for failures or just return in case kzalloc fails.

Regards
Matthias
Matthias,

Thank you for pointing this out -- I am pushing up a fix right now.

Mauro,

Please pull from:

http://linuxtv.org/hg/~mkrufky/lgdt3305

for the following fix, thanks to Matthias:

- lgdt3305: avoid OOPS in error path of lgdt3305_attach

lgdt3305.c |    1 -
1 file changed, 1 deletion(-)

Regards,

Mike
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to