On Thu, 03 Dec 2015 22:08:51 +0100, Subhransu S. Prusty wrote: > static int hdac_hdmi_dev_remove(struct hdac_ext_device *edev) > { > + struct hdac_hdmi_priv *hdmi = edev->private_data; > + struct hdac_hdmi_pin *pin; > + struct hdac_hdmi_cvt *cvt; > + > snd_soc_unregister_codec(&edev->hdac.dev); > > + list_for_each_entry(cvt, &hdmi->cvt_list, head) { > + list_del(&cvt->head); > + kfree(cvt); > + } > + > + list_for_each_entry(pin, &hdmi->pin_list, head) { > + list_del(&pin->head); > + kfree(pin); > + } These must be list_for_each_entry_safe(). Takashi _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel