linux-kernel - Re: [PATCH] ALSA: use designated initializers

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]

Date:   Sat, 17 Dec 2016 19:17:13 +0900
From:   Takashi Sakamoto <o-takashi@...amocchi.jp>
To:     Kees Cook <keescook@...omium.org>, linux-kernel@...r.kernel.org
Cc:     alsa-devel@...a-project.org, Takashi Iwai <tiwai@...e.com>
Subject: Re: [PATCH] ALSA: use designated initializers

On Dec 17 2016 09:59, Kees Cook wrote:
> Prepare to mark sensitive kernel structures for randomization by making
> sure they're using designated initializers. These were identified during
> allyesconfig builds of x86, arm, and arm64, with most initializer fixes
> extracted from grsecurity.
>
> Signed-off-by: Kees Cook <keescook@...omium.org>
> ---
>  sound/synth/emux/emux_seq.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)

Reviewed-by: Takashi Sakamoto <o-takashi@...amocchi.jp>

> diff --git a/sound/synth/emux/emux_seq.c b/sound/synth/emux/emux_seq.c
> index a0209204ae48..55579f6b8cb2 100644
> --- a/sound/synth/emux/emux_seq.c
> +++ b/sound/synth/emux/emux_seq.c
> @@ -33,13 +33,13 @@ static int snd_emux_unuse(void *private_data, struct snd_seq_port_subscribe *inf
>   * MIDI emulation operators
>   */
>  static struct snd_midi_op emux_ops = {
> -	snd_emux_note_on,
> -	snd_emux_note_off,
> -	snd_emux_key_press,
> -	snd_emux_terminate_note,
> -	snd_emux_control,
> -	snd_emux_nrpn,
> -	snd_emux_sysex,
> +	.note_on = snd_emux_note_on,
> +	.note_off = snd_emux_note_off,
> +	.key_press = snd_emux_key_press,
> +	.note_terminate = snd_emux_terminate_note,
> +	.control = snd_emux_control,
> +	.nrpn = snd_emux_nrpn,
> +	.sysex = snd_emux_sysex,
>  };

Regards

Takashi Sakamoto