[alsa-devel] [PATCH - JACK plugin v2 0/2] jack: Support to connect multiple JACK ports with same ALSA channel
Takashi Iwai
tiwai at suse.de
Tue Jan 29 16:28:13 CET 2019
On Tue, 29 Jan 2019 15:30:54 +0100,
<twischer at de.adit-jv.com> wrote:
>
>
> On 1/29/19 14:45, Takashi Iwai wrote:
> > name is already dereferenced in strlen(), so it's too late here.
> > i.e. name_size has to be evaluated after this NULL check, if any.
> > Or just drop this NULL check. It's an internal function and the name
> > is never NULL, practically seen.
>
> I have removed it.
>
> > name_size is the exact size of string + 1, so this is equivalent with
> > strcpy().
>
> I have added a comment and I am using strcpy(), now.
These aren't really good cover letter :)
(FWIW, the patch 0/N should be a cover letter that describes the patch
set.)
In anyway, I applied the latest patches now.
thanks,
Takashi
More information about the Alsa-devel
mailing list