5520 - _PR_MD_LSEEK64 tests for the failure of SetFilePointer incorrectly.

Status: VERIFIED FIXED

(NSPR :: NSPR, defect, P3)

Description

[Wan-Teh Chang]

In the Win95 version of _PR_MD_LSEEK64 (w95io.c),
we only check the return value of SetFilePointer
to see if it fails.  That's not sufficient for
a large file.  When the third argument (lpDistanceToMoveHigh)
for SetFilePointer is not NULL, if SetFilePointer
returns 0xFFFFFFFF, we also need to call GetLastError
to find out if SetFilePointer really fails.

The NT version of _PR_MD_LSEEK64 (ntio.c) does this
correctly.

Comment 1

[Wan-Teh Chang]

Fixed.  I also rewrote the code for converting between
the 64-bit offset and its two 32-bit components, following
the example in MSDN documentation of SetFilePointer
(http://msdn.microsoft.com/library/sdkdoc/winbase/filesio_8iwi.htm).
/cvsroot/mozilla/nsprpub/pr/src/md/windows/w95io.c, revision 3.8.

Comment 2

[Terry Weissman]

NSPR now has its own Bugzilla product.  Moving this bug to the NSPR product.

Comment 3

[phillip]

this fix is not visible to qa. if you think this bug should be reopened,
please do so.

will mark as verified.