On Thu, 04 Oct 2018 01:45:41 +0200, Andre Guedes wrote: > > +static int aaf_close(snd_pcm_ioplug_t *io) > +{ > + snd_pcm_aaf_t *aaf = io->private_data; > + > + if (!aaf) > + return -EBADFD; > + > + free(aaf); > + aaf = NULL; This makes no sense. The function can be simplified to static int aaf_close(snd_pcm_ioplug_t *io) { free(io->private_data); return 0; } thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel