Fix an off-by-one in the zip filename reading code. It's pretty harmless, and...

chrome/common/zip.cc

 // Copyright (c) 2009 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 "chrome/common/zip.h"
 
 #include "base/file_util.h"
 #include "base/string_util.h"
 #include "net/base/file_stream.h"
 #include "third_party/zlib/contrib/minizip/unzip.h"
 #include "third_party/zlib/contrib/minizip/zip.h"
 #if defined(OS_WIN)
 #include "third_party/zlib/contrib/minizip/iowin32.h"
 #endif
 
 static const int kZipMaxPath = 256;
 static const int kZipBufSize = 8192;
 
 // Extract the 'current' selected file from the zip into dest_dir.
 // Output filename is stored in out_file.  Returns true on success.
 static bool ExtractCurrentFile(unzFile zip_file,
                                const FilePath& dest_dir) {
   char filename_inzip[kZipMaxPath] = {0};
   unz_file_info file_info;
   int err = unzGetCurrentFileInfo(zip_file, &file_info, filename_inzip,
-                                  sizeof(filename_inzip), NULL, 0, NULL, 0);
+                                  sizeof(filename_inzip) - 1, NULL, 0, NULL, 0);
   if (err != UNZ_OK)
     return false;
   if (filename_inzip[0] == '\0')
     return false;
 
   err = unzOpenCurrentFile(zip_file);
   if (err != UNZ_OK)
     return false;
 
   FilePath::StringType filename;
@@ skipping 255 matching lines @@
     }
   }
 
   if (ZIP_OK != zipClose(zip_file, NULL)) {  // global comment
     LOG(ERROR) << "Error closing zip file " << dest_file_str;
     return false;
   }
 
   return success;
 }