Added more error checking for audio packets.

remoting/codec/audio_decoder_verbatim.cc

Index: remoting/codec/audio_decoder_verbatim.cc
diff --git a/remoting/codec/audio_decoder_verbatim.cc b/remoting/codec/audio_decoder_verbatim.cc
index 903cb3f298b824a34928ac795c8566521321f9fa..43919c6c99f1f95cf375d3ddc0bbbb9651cd5dbf 100644
--- a/remoting/codec/audio_decoder_verbatim.cc
+++ b/remoting/codec/audio_decoder_verbatim.cc
@@ -21,7 +21,8 @@ scoped_ptr<AudioPacket> AudioDecoderVerbatim::Decode(
   if ((packet->encoding() != AudioPacket::ENCODING_RAW) ||
       (packet->data_size() != 1) ||
       (packet->sampling_rate() == AudioPacket::SAMPLING_RATE_INVALID) ||
-      (packet->bytes_per_sample() == AudioPacket::BYTES_PER_SAMPLE_INVALID)) {
+      (packet->bytes_per_sample() == AudioPacket::BYTES_PER_SAMPLE_INVALID) ||

[Sergey Ulanov, 2012/08/20 22:26:20]

here you want to check that bytes_per_sample() == 2 as that's the only format
currently supported by the player.

[kxing, 2012/08/20 22:48:22]

Done.

+      (packet->channels() == AudioPacket::CHANNELS_INVALID)) {

[Sergey Ulanov, 2012/08/20 22:26:20]

check that we have two channels, not only that the number of channels is valid.
If the host sends us Mono stream the current player implementation will try to
play it as stereo.

[kxing, 2012/08/20 22:48:22]

Done.

     return scoped_ptr<AudioPacket>();
   }
   return packet.Pass();