Fix allocation size in CricketToJavaI420Frame, taking stride into account.

webrtc/api/java/jni/peerconnection_jni.cc

 /*
  *  Copyright 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 753 matching lines @@
     jint* strides_array = jni()->GetIntArrayElements(strides, NULL);
     strides_array[0] = frame->video_frame_buffer()->StrideY();
     strides_array[1] = frame->video_frame_buffer()->StrideU();
     strides_array[2] = frame->video_frame_buffer()->StrideV();
     jni()->ReleaseIntArrayElements(strides, strides_array, 0);
     jobjectArray planes = jni()->NewObjectArray(3, *j_byte_buffer_class_, NULL);
     jobject y_buffer = jni()->NewDirectByteBuffer(
         const_cast<uint8_t*>(frame->video_frame_buffer()->DataY()),
         frame->video_frame_buffer()->StrideY() *
             frame->video_frame_buffer()->height());
-    size_t chroma_size =
-      ((frame->width() + 1) / 2) * ((frame->height() + 1) / 2);
+    size_t chroma_height = (frame->height() + 1) / 2;
     jobject u_buffer = jni()->NewDirectByteBuffer(
         const_cast<uint8_t*>(frame->video_frame_buffer()->DataU()),
-        chroma_size);
+        frame->video_frame_buffer()->StrideU() * chroma_height);
     jobject v_buffer = jni()->NewDirectByteBuffer(
         const_cast<uint8_t*>(frame->video_frame_buffer()->DataV()),
-        chroma_size);
+        frame->video_frame_buffer()->StrideV() * chroma_height);
+
     jni()->SetObjectArrayElement(planes, 0, y_buffer);
     jni()->SetObjectArrayElement(planes, 1, u_buffer);
     jni()->SetObjectArrayElement(planes, 2, v_buffer);
     return jni()->NewObject(
         *j_frame_class_, j_i420_frame_ctor_id_,
         frame->width(), frame->height(),
         static_cast<int>(frame->rotation()),
         strides, planes, javaShallowCopy(frame));
   }
 
@@ skipping 1089 matching lines @@
 JOW(void, VideoRenderer_nativeCopyPlane)(
     JNIEnv *jni, jclass, jobject j_src_buffer, jint width, jint height,
     jint src_stride, jobject j_dst_buffer, jint dst_stride) {
   size_t src_size = jni->GetDirectBufferCapacity(j_src_buffer);
   size_t dst_size = jni->GetDirectBufferCapacity(j_dst_buffer);
   RTC_CHECK(src_stride >= width) << "Wrong source stride " << src_stride;
   RTC_CHECK(dst_stride >= width) << "Wrong destination stride " << dst_stride;
   RTC_CHECK(src_size >= src_stride * height)
       << "Insufficient source buffer capacity " << src_size;
   RTC_CHECK(dst_size >= dst_stride * height)
-      << "Isufficient destination buffer capacity " << dst_size;
+      << "Insufficient destination buffer capacity " << dst_size;
   uint8_t *src =
       reinterpret_cast<uint8_t*>(jni->GetDirectBufferAddress(j_src_buffer));
   uint8_t *dst =
       reinterpret_cast<uint8_t*>(jni->GetDirectBufferAddress(j_dst_buffer));
   if (src_stride == dst_stride) {
     memcpy(dst, src, src_stride * height);
   } else {
     for (int i = 0; i < height; i++) {
       memcpy(dst, src, width);
       src += src_stride;
@@ skipping 307 matching lines @@
   return JavaStringFromStdString(
       jni,
       reinterpret_cast<RtpReceiverInterface*>(j_rtp_receiver_pointer)->id());
 }
 
 JOW(void, RtpReceiver_free)(JNIEnv* jni, jclass, jlong j_rtp_receiver_pointer) {
   reinterpret_cast<RtpReceiverInterface*>(j_rtp_receiver_pointer)->Release();
 }
 
 }  // namespace webrtc_jni