Re: [PATCH v1 7/8] ALSA: hda/tas2781: Add tas2781 SPI-based driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Mar 26, 2024 at 10:13:06AM -0500, Pierre-Louis Bossart wrote:


...

> memory leaks for each of those goto out;
> 
> You need to use different labels and free-up what was allocated before.

Good point, but better to just use cleanup.h from day 1, it makes this code
shrink even more.

...

> I'll stop the review here.

He-he :-)

-- 
With Best Regards,
Andy Shevchenko





[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux