linux-kernel - Re: [PATCH 1/1] ASoc: simple-card-utils: add _

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <7bb40eff-d1ae-4739-0396-6f24089416f5@m4x.org>
Date:   Mon, 22 Aug 2016 15:06:05 +0200
From:   Nicolas Iooss <nicolas.iooss_linux@....org>
To:     Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] ASoc: simple-card-utils: add __printf attribute

Hello,

After I sent the following patch a few weeks ago, I have not received
any feedback. Could you please review it?

Thanks,
Nicolas

On 06/08/16 11:58, Nicolas Iooss wrote:
> asoc_simple_card_set_dailink_name() uses devm_kvasprintf() to format
> some of its arguments.  Adding a __printf attribute to this function
> makes it possible to detect at compile-time errors related to format
> strings.
> 
> Signed-off-by: Nicolas Iooss <nicolas.iooss_linux@....org>
> ---
>  include/sound/simple_card_utils.h | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/include/sound/simple_card_utils.h b/include/sound/simple_card_utils.h
> index 86088aed9002..3207b1a70d38 100644
> --- a/include/sound/simple_card_utils.h
> +++ b/include/sound/simple_card_utils.h
> @@ -27,6 +27,7 @@ int asoc_simple_card_parse_daifmt(struct device *dev,
>  				  struct device_node *codec,
>  				  char *prefix,
>  				  unsigned int *retfmt);
> +__printf(3, 4)
>  int asoc_simple_card_set_dailink_name(struct device *dev,
>  				      struct snd_soc_dai_link *dai_link,
>  				      const char *fmt, ...);
>