From 49273ffa792293d1b8df17ae90a4f11cd6b6c401 Mon Sep 17 00:00:00 2001 From: "braveyao@webrtc.org" Date: Mon, 14 Jan 2013 01:52:26 +0000 Subject: [PATCH] logical 'and' of mutually exclusive tests is always false in ViECodecImpl::CodecValid() BUG = Issue1283 Review URL: https://webrtc-codereview.appspot.com/1013008 git-svn-id: http://webrtc.googlecode.com/svn/trunk@3371 4adac7df-926f-26a2-2b94-8c16560cd09d --- webrtc/video_engine/vie_codec_impl.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/webrtc/video_engine/vie_codec_impl.cc b/webrtc/video_engine/vie_codec_impl.cc index 712bbf896e..3ae15c70e4 100644 --- a/webrtc/video_engine/vie_codec_impl.cc +++ b/webrtc/video_engine/vie_codec_impl.cc @@ -748,7 +748,7 @@ bool ViECodecImpl::CodecValid(const VideoCodec& video_codec) { return false; } - if (video_codec.plType == 0 && video_codec.plType > 127) { + if (video_codec.plType == 0 || video_codec.plType > 127) { WEBRTC_TRACE(kTraceError, kTraceVideo, -1, "Invalid codec payload type: %d", video_codec.plType); return false;