shithub: libvpx

Download patch

ref: ba159a3b0a80ff97e17ad946da4b7fef4a7cfe5a
parent: d743bd04d5c9261244bbe2838dc8014d770b621d
author: James Zern <jzern@google.com>
date: Fri Feb 28 14:58:19 EST 2014

nestegg: fix track_number uint64->uint32 warnings

ne_read_block/ne_find_cue_position_for_track/nestegg_get_cue_point
in the use of ne_map_track_number_to_index
+ add a check to ensure it doesn't exceed the type bounds

fixes:
./third_party/nestegg/src/nestegg.c|1322| warning C4244: 'function' :
conversion from 'uint64_t' to 'unsigned int', possible loss of data

Change-Id: I3703d739dcf9a2d4d8e2b704e957e5e3fd80dca0

--- a/third_party/nestegg/README.webm
+++ b/third_party/nestegg/README.webm
@@ -15,3 +15,4 @@
   fixes:
 nestegg.c|975 col 6| warning: ‘r’ may be used uninitialized in this function [-Wuninitialized]
 - add ne_get_uint32 convenience function
+- fix track_number uint64->uint32 warnings
--- a/third_party/nestegg/src/nestegg.c
+++ b/third_party/nestegg/src/nestegg.c
@@ -1260,7 +1260,7 @@
   if (r != 1)
     return r;
 
-  if (track_number == 0)
+  if (track_number == 0 || (unsigned int)track_number != track_number)
     return -1;
 
   consumed += length;
@@ -1333,7 +1333,7 @@
   if (total > block_size)
     return -1;
 
-  if (ne_map_track_number_to_index(ctx, track_number, &track) != 0)
+  if (ne_map_track_number_to_index(ctx, (unsigned int)track_number, &track) != 0)
     return -1;
 
   entry = ne_find_track_entry(ctx, track);
@@ -1858,7 +1858,7 @@
         if (ne_get_uint(pos->track, &track_number) != 0)
           return -1;
 
-        if (ne_map_track_number_to_index(ctx, track_number, &track_index) != 0)
+        if (ne_map_track_number_to_index(ctx, (unsigned int)track_number, &track_index) != 0)
           return -1;
 
         if (track_index == track) {
--