Fixes for re-enabling more MSVC level 4 warnings: content/browser/ edition

content/browser/renderer_host/p2p/socket_host.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/p2p/socket_host.h"
 
 #include "base/sys_byteorder.h"
 #include "content/browser/renderer_host/p2p/socket_host_tcp.h"
 #include "content/browser/renderer_host/p2p/socket_host_tcp_server.h"
 #include "content/browser/renderer_host/p2p/socket_host_udp.h"
@@ skipping 203 matching lines @@
   }
 
   UpdateRtpAuthTag(start, rtp_length, options);
   return true;
 }
 
 bool GetRtpPacketStartPositionAndLength(const char* packet,
                                         int length,
                                         int* rtp_start_pos,
                                         int* rtp_packet_length) {
-  int rtp_begin, rtp_length;
+  int rtp_begin;
+  int rtp_length = 0;
   if (IsTurnChannelData(packet)) {
     // Turn Channel Message header format.
     //   0                   1                   2                   3
     //  0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
     // +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
     // |         Channel Number        |            Length             |
     // +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
     // |                                                               |
     // /                       Application Data                        /
     // /                                                               /
@@ skipping 72 matching lines @@
       return false;
     }
 
   } else {
     // This is a raw RTP packet.
     rtp_begin = 0;
     rtp_length = length;
   }
 
   // Making sure we have a valid RTP packet at the end.
-  if (!(rtp_length < kMinRtpHdrLen) &&
+  if ((rtp_length >= kMinRtpHdrLen) &&
       IsRtpPacket(packet + rtp_begin, rtp_length) &&
       ValidateRtpHeader(packet + rtp_begin, rtp_length, NULL)) {
     *rtp_start_pos = rtp_begin;
     *rtp_packet_length = rtp_length;
     return true;
   }
   return false;
 }
 
 // ValidateRtpHeader must be called before this method to make sure, we have
@@ skipping 248 matching lines @@
   BrowserThread::PostTask(BrowserThread::UI,
                           FROM_HERE,
                           base::Bind(packet_dump_callback_,
                                      Passed(&packet_header),
                                      header_length,
                                      packet_length,
                                      incoming));
 }
 
 }  // namespace content