On Monday 02 of April 2012, Arnaud Versini wrote:
Hi,
It was only to make sure we don't overread the array, and because of the
valgrind message.
Sorry
 No need to be sorry, handling the .doc format is nothing trivial :). I was 
only curious if you did the patch based only on valgrind or if you had some 
documents where the sizes didn't exactly match what the spec says.
 Miklos has already fixed Read_UL(), but your patch lists more places. Could 
you check why those places needed the change and make them read the proper 
sizes according to what the .doc spec says?
-- 
 Lubos Lunak
 l.lunak@suse.cz
Context
   
 
  Privacy Policy |
  
Impressum (Legal Info) |
  
Copyright information: Unless otherwise specified, all text and images
  on this website are licensed under the
  
Creative Commons Attribution-Share Alike 3.0 License.
  This does not include the source code of LibreOffice, which is
  licensed under the Mozilla Public License (
MPLv2).
  "LibreOffice" and "The Document Foundation" are
  registered trademarks of their corresponding registered owners or are
  in actual use as trademarks in one or more countries. Their respective
  logos and icons are also subject to international copyright laws. Use
  thereof is explained in our 
trademark policy.