[lkml]   [2019]   [Dec]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] media: go7007: fix a miss of snd_card_free
On Fri, Dec 6, 2019 at 10:11 PM Markus Elfring <> wrote:
> > go7007_snd_init() misses a snd_card_free() in an error path.
> > Add the missed call to fix it.
> How do you think about to take another look also at the previous update suggestion
> “[PATCH 5/6] [media] go7007: Use common error handling code in go7007_snd_init()”?

I think using goto is really much better than freeing after every error.
I will send a new version if others also agree with this pattern.


> Regards,
> Markus

 \ /
  Last update: 2019-12-06 15:38    [W:0.035 / U:5.484 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site