| From dfb00a56935186171abb5280b3407c3f910011f1 Mon Sep 17 00:00:00 2001 |
| From: Takashi Sakamoto <o-takashi@sakamocchi.jp> |
| Date: Fri, 14 Apr 2017 12:43:01 +0900 |
| Subject: ALSA: firewire-lib: fix inappropriate assignment between signed/unsigned type |
| |
| From: Takashi Sakamoto <o-takashi@sakamocchi.jp> |
| |
| commit dfb00a56935186171abb5280b3407c3f910011f1 upstream. |
| |
| An abstraction of asynchronous transaction for transmission of MIDI |
| messages was introduced in Linux v4.4. Each driver can utilize this |
| abstraction to transfer MIDI messages via fixed-length payload of |
| transaction to a certain unit address. Filling payload of the transaction |
| is done by callback. In this callback, each driver can return negative |
| error code, however current implementation assigns the return value to |
| unsigned variable. |
| |
| This commit changes type of the variable to fix the bug. |
| |
| Reported-by: Julia Lawall <Julia.Lawall@lip6.fr> |
| Fixes: 585d7cba5e1f ("ALSA: firewire-lib: add helper functions for asynchronous transactions to transfer MIDI messages") |
| Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp> |
| Signed-off-by: Takashi Iwai <tiwai@suse.de> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| sound/firewire/lib.h | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/sound/firewire/lib.h |
| +++ b/sound/firewire/lib.h |
| @@ -45,7 +45,7 @@ struct snd_fw_async_midi_port { |
| |
| struct snd_rawmidi_substream *substream; |
| snd_fw_async_midi_port_fill fill; |
| - unsigned int consume_bytes; |
| + int consume_bytes; |
| }; |
| |
| int snd_fw_async_midi_port_init(struct snd_fw_async_midi_port *port, |