Fix HEVC depacketizer issues. (#185)

- Incorrect VPS ID.
- FindFrames doesn't return packets.

Author: jianjunz

common_video/h265/h265_pps_parser.cc

@@ -202,6 +202,10 @@ absl::optional<H265PpsParser::PpsState> H265PpsParser::ParseInternal(
   // slice_segment_header_extension_present_flag: u(1)
   reader.ConsumeBits(1);
 
+  if (!reader.Ok()) {
+    return absl::nullopt;
+  }
+
   return pps;
 }
 

common_video/h265/h265_sps_parser.cc

@@ -395,6 +395,10 @@ absl::optional<H265SpsParser::SpsState> H265SpsParser::ParseSpsInternal(
     sps.height -= sub_height_c * (conf_win_top_offset + conf_win_bottom_offset);
   }
 
+  if (!reader.Ok()) {
+    return absl::nullopt;
+  }
+
   return OptionalSps(sps);
 }
 

common_video/h265/h265_vps_parser.cc

@@ -43,7 +43,11 @@ absl::optional<H265VpsParser::VpsState> H265VpsParser::ParseInternal(
   VpsState vps;
 
   // vps_video_parameter_set_id: u(4)
-  vps.id = reader.Read<uint32_t>();
+  vps.id = reader.ReadBits(4);
+
+  if (!reader.Ok()) {
+    return absl::nullopt;
+  }
 
   return vps;
 }

modules/video_coding/packet_buffer.cc

@@ -447,7 +447,7 @@ std::vector<std::unique_ptr<PacketBuffer::Packet>> PacketBuffer::FindFrames(
         }
       }
 #endif
-      if (is_h264 || full_frame_found) {
+      if (is_h264 || is_h265 || full_frame_found) {
         const uint16_t end_seq_num = seq_num + 1;
         // Use uint16_t type to handle sequence number wrap around case.
         uint16_t num_packets = end_seq_num - start_seq_num;