Re: questionable code

[Robert Shearman]

Eric Pouech wrote:
> *** dlls/ole32/hglobalstream.c line 602
> the test "if (newPosition.QuadPart < 0)" is always false as QuadPart 
> is unsigned

Before the call to RtlLargeIntegerAdd we need to add a line with:
if ((dlibMove.QuadPart < 0) && (newMove < -dlibMove.QuadPart)) return 
STG_E_INVALIDFUNCTION;

And remove the offending < 0 line.

You can submit a patch for me, or wait until Monday and I'll submit one.

Rob