ALSA: core: don't return uninitialized snd_compr_tstamp
authorRichard Fitzgerald <rf@opensource.wolfsonmicro.com>
Mon, 11 Feb 2013 13:44:53 +0000 (13:44 +0000)
committerTakashi Iwai <tiwai@suse.de>
Mon, 11 Feb 2013 14:03:48 +0000 (15:03 +0100)
commit17ac8e5c6d3478dcfeb75ed5716ca7e5cee612f0
treedbf86ce267ccd64c980bc496145e596a97769520
parentf664417e23192087bb9bdafdff80e04104994cc0
ALSA: core: don't return uninitialized snd_compr_tstamp

The snd_compr_update_tstamp() can only fill in the snd_compr_tstamp
if the codec implements the pointer() function. If that happened
the code was previously returning uninitialized garbage in the
tstamp because it wasn't initialized anywhere.

This change zero-fills the tstamp in the two places it is used
before calling snd_compr_update_tstamp(), and also has
snd_compr_update_tstamp() return an error indication if it
can't provide a tstamp. For the case of snd_compr_calc_avail()
it ignores this error because we still need to return info on
the available buffer space even if we can't provide tstamp
info - when the tstamp is not valid all fields are now
guaranteed to be zero.

Signed-off-by: Richard Fitzgerald <rf@opensource.wolfsonmicro.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/core/compress_offload.c