> > + ret = snd_soc_jack_new(&snd_soc_sdp3430, "SDP3430 headset jack", > > + SND_JACK_HEADSET, hs_jack); > > + if (ret) > > + return ret; > This leaks the jack. If hs_jack is declared as a direct static variable instead, then no mem leaks, isn't it? Or do you mean to clean the hs_jack if any error? -Misa _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel