On Tue, Mar 15, 2011 at 8:07 AM, Dan Carpenter <erro...@gmail.com> wrote: > In the earlier check we assumed that "obj" could be NULL. I looked at > some of the other places that call evaluate_object() and they check > for NULL as well. > > Signed-off-by: Dan Carpenter <erro...@gmail.com> > --- > Can obj actually be NULL here? Perhaps a better fix would be to remove > the first check.
Yes I think it can. > diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c > index a038595..efc776c 100644 > --- a/drivers/platform/x86/asus-wmi.c > +++ b/drivers/platform/x86/asus-wmi.c > @@ -1343,7 +1343,7 @@ static int show_call(struct seq_file *m, void *data) > else > seq_printf(m, "%#x(%#x, %#x) = t:%d\n", asus->debug.method_id, > asus->debug.dev_id, asus->debug.ctrl_param, > - obj->type); > + obj ? obj->type : -1); Seems ok, I may change that later if we need to support more types. Acked-by: Corentin Chary <corentin.ch...@gmail.com> -- Corentin Chary http://xf.iksaif.net -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
