sound: fix check for return value in snd_pcm_hw_refine

'params' is a pointer and looking at the code this probably should be a check for ioctl return value. Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl> Signed-off-by: Takashi Iwai <tiwai@suse.de>

sound: fix check for return value in snd_pcm_hw_refine
'params' is a pointer and looking at the code this probably should be a check for ioctl return value. Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Mariusz Kozlowski · Takashi Iwai
1 parent 4f64e15019
Showing 1 changed file with 1 additions and 1 deletions Side-by-side Diff
sound/core/pcm_native.c
@@ -320,7 +320,7 @@
                     snd_mask_max(&params->masks[SNDRV_PCM_HW_PARAM_CHANNELS])) {
 			changed = substream->ops->ioctl(substream,
 					SNDRV_PCM_IOCTL1_FIFO_SIZE, params);
-			if (params < 0)
+			if (changed < 0)
 				return changed;
 		}
 	}
...	...	@@ -320,7 +320,7 @@
320	320	snd_mask_max(&params->masks[SNDRV_PCM_HW_PARAM_CHANNELS])) {
321	321	changed = substream->ops->ioctl(substream,
322	322	SNDRV_PCM_IOCTL1_FIFO_SIZE, params);
323		- if (params < 0)
	323	+ if (changed < 0)
324	324	return changed;
325	325	}
326	326	}