Comment 2 for bug 724102

Revision history for this message
Neil Mayhew (neil.mayhew) wrote :

This problem should have produced an error message but didn't, which caused me to waste a lot of time trying to figure out why the 8GB cards I had created didn't work. The reason it didn't produce an error is that the return status from SetFilePointer isn't checked like it should be.

This patch adds these checks in addition to using 64-bit file offsets as Jeff L's patch does. Also, it uses SetFilePointerEx which is a slightly neater way to handle 64-bit offsets.

Is there any chance this fix could be merged and a new release made? I am recommending this utility to my users but I'd much rather they got an official released copy instead of getting a patched one from me.