Am 07.11.2017 um 14:44 schrieb Mauro Carvalho Chehab: > diff --git a/drivers/media/dvb-core/dvb_frontend.c b/drivers/media/dvb-core/dvb_frontend.c > index d485d5f6cc88..3ad83359098b 100644 > --- a/drivers/media/dvb-core/dvb_frontend.c > +++ b/drivers/media/dvb-core/dvb_frontend.c > @@ -150,11 +150,8 @@ static void __dvb_frontend_free(struct dvb_frontend *fe) > > dvb_frontend_invoke_release(fe, fe->ops.release); > > - if (!fepriv) > - return; > - > - kfree(fepriv); > - fe->frontend_priv = NULL; > + if (fepriv) > + kfree(fepriv); I think the condition is redundant and should be removed. kfree(NULL) is fine. > } > > static void dvb_frontend_free(struct kref *ref) > Regards Matthias
