Message ID | 20180928212727.15151-2-andre.guedes@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | Fix leaks on multiple plugins | expand |
diff --git a/a52/pcm_a52.c b/a52/pcm_a52.c index 29ce45f..e431fd0 100644 --- a/a52/pcm_a52.c +++ b/a52/pcm_a52.c @@ -978,7 +978,7 @@ SND_PCM_PLUGIN_DEFINE_FUNC(a52) if ((err = a52_set_hw_constraint(rec)) < 0) { snd_pcm_ioplug_delete(&rec->io); - return err; + goto error; } *pcmp = rec->io.pcm;
If a52_set_hw_constraint() returns error, we leak 'rec' and 'rec->slave'. This patch fixes the issue by jumping to 'error' label where the proper clean up is already done. Signed-off-by: Andre Guedes <andre.guedes@intel.com> --- a52/pcm_a52.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)