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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html