Ros-diffs March 2012

ros-diffs@reactos.org

22 participants
351 discussions

[sir_richard] 55999: [NTOS]: Drop a reference count to the page table in MiDeleteSystemPageableVm as ARM3 now owns PDEs. This helps out yet again to free a little bit of extra RAM up.

by sir_richard＠svn.reactos.org

Author: sir_richard Date: Sun Mar 4 17:42:56 2012 New Revision: 55999 URL: http://svn.reactos.org/svn/reactos?rev=55999&view=rev Log: [NTOS]: Drop a reference count to the page table in MiDeleteSystemPageableVm as ARM3 now owns PDEs. This helps out yet again to free a little bit of extra RAM up. Modified: trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c trunk/reactos/ntoskrnl/mm/ARM3/virtual.c Modified: trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/pagfault.… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c [iso-8859-1] Sun Mar 4 17:42:56 2012 @@ -371,9 +371,10 @@ IN PMMPFN Pfn1) { MMPTE TempPte; - PMMPTE OriginalPte; + PMMPTE OriginalPte, PageTablePte; ULONG_PTR Protection; PFN_NUMBER PageFrameIndex; + PMMPFN Pfn2; /* Must be called with an valid prototype PTE, with the PFN lock held */ ASSERT(KeGetCurrentIrql() == DISPATCH_LEVEL); @@ -386,7 +387,12 @@ Pfn1 = MiGetPfnEntry(PageFrameIndex); Pfn1->u3.e1.PrototypePte = 1; - /* FIXME: Increment the share count for the page table */ + /* Increment the share count for the page table */ + // FIXME: This doesn't work because we seem to bump the sharecount to two, and MiDeletePte gets annoyed and ASSERTs. + // This could be beause MiDeletePte is now being called from strange code in Rosmm + PageTablePte = MiAddressToPte(PointerPte); + Pfn2 = MiGetPfnEntry(PageTablePte->u.Hard.PageFrameNumber); + //Pfn2->u2.ShareCount++; /* Check where we should be getting the protection information from */ if (PointerPte->u.Soft.PageFileHigh == MI_PTE_LOOKUP_NEEDED) Modified: trunk/reactos/ntoskrnl/mm/ARM3/virtual.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/virtual.c… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] Sun Mar 4 17:42:56 2012 @@ -116,8 +116,7 @@ { PFN_COUNT ActualPages = 0; PETHREAD CurrentThread = PsGetCurrentThread(); - PMMPFN Pfn1; - //PMMPFN Pfn2; + PMMPFN Pfn1, Pfn2; PFN_NUMBER PageFrameIndex, PageTableIndex; KIRQL OldIrql; ASSERT(KeGetCurrentIrql() <= APC_LEVEL); @@ -150,7 +149,7 @@ /* Get the page table entry */ PageTableIndex = Pfn1->u4.PteFrame; - //Pfn2 = MiGetPfnEntry(PageTableIndex); + Pfn2 = MiGetPfnEntry(PageTableIndex); /* Lock the PFN database */ OldIrql = KeAcquireQueuedSpinLock(LockQueuePfnLock); @@ -160,10 +159,7 @@ MiDecrementShareCount(Pfn1, PageFrameIndex); /* Decrement the page table too */ - DPRINT("FIXME: ARM3 should decrement the pool PDE refcount for: %p\n", PageTableIndex); - #if 0 // ARM3: Dont't trust this yet MiDecrementShareCount(Pfn2, PageTableIndex); - #endif /* Release the PFN database */ KeReleaseQueuedSpinLock(LockQueuePfnLock, OldIrql); @@ -293,8 +289,7 @@ /* There should only be 1 shared reference count */ ASSERT(Pfn1->u2.ShareCount == 1); - /* FIXME: Drop the reference on the page table. For now, leak it until RosMM is gone */ - //DPRINT1("Dropping a ref...\n"); + /* Drop the reference on the page table. */ MiDecrementShareCount(MiGetPfnEntry(Pfn1->u4.PteFrame), Pfn1->u4.PteFrame); /* Mark the PFN for deletion and dereference what should be the last ref */

12 years, 8 months

[tkreuzer] 55998: [GENDIB] - Generate ansi style indentation - Fix signed/unsigned MSVC warnings in generated code

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sun Mar 4 13:33:07 2012 New Revision: 55998 URL: http://svn.reactos.org/svn/reactos?rev=55998&view=rev Log: [GENDIB] - Generate ansi style indentation - Fix signed/unsigned MSVC warnings in generated code Modified: trunk/reactos/tools/gendib/gendib.c Modified: trunk/reactos/tools/gendib/gendib.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/gendib/gendib.c?rev=… ============================================================================== --- trunk/reactos/tools/gendib/gendib.c [iso-8859-1] (original) +++ trunk/reactos/tools/gendib/gendib.c [iso-8859-1] Sun Mar 4 13:33:07 2012 @@ -177,14 +177,11 @@ unsigned n; va_list Args; - if (NULL != strchr(Fmt, '{') && 0 != Indent) - { - Indent += 2; - } - else if (NULL != strchr(Fmt, '}')) - { - Indent -= 2; - } + if (strchr(Fmt, '}')) + { + Indent -= 4; + } + if (AtBOL) { for (n = 0; n < Indent; n++) @@ -198,14 +195,11 @@ vfprintf(Out, Fmt, Args); va_end(Args); - if (NULL != strchr(Fmt, '{')) - { - Indent += 2; - } - else if (NULL != strchr(Fmt, '}') && 0 != Indent) - { - Indent -= 2; - } + if (strchr(Fmt, '{')) + { + Indent += 4; + } + AtBOL = '\n' == Fmt[strlen(Fmt) - 1]; } @@ -460,7 +454,7 @@ else { Output(Out, "LeftCount = (ULONG_PTR) DestBase & 0x03;\n"); - Output(Out, "if (BltInfo->DestRect.right - BltInfo->DestRect.left < " + Output(Out, "if ((ULONG)(BltInfo->DestRect.right - BltInfo->DestRect.left) < " "LeftCount)\n"); Output(Out, "{\n"); Output(Out, "LeftCount = BltInfo->DestRect.right - " @@ -800,7 +794,7 @@ Output(Out, "ULONG i;\n"); if (RopInfo->UsesPattern) { - Output(Out, "ULONG PatternX =0, PatternY = 0, BasePatternX = 0;\n"); + Output(Out, "LONG PatternX =0, PatternY = 0, BasePatternX = 0;\n"); } First = 1; if (RopInfo->UsesSource)

12 years, 8 months

[tkreuzer] 55997: [GENDIB] Fix formatting

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sun Mar 4 13:06:02 2012 New Revision: 55997 URL: http://svn.reactos.org/svn/reactos?rev=55997&view=rev Log: [GENDIB] Fix formatting Modified: trunk/reactos/tools/gendib/gendib.c Modified: trunk/reactos/tools/gendib/gendib.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/gendib/gendib.c?rev=… ============================================================================== --- trunk/reactos/tools/gendib/gendib.c [iso-8859-1] (original) +++ trunk/reactos/tools/gendib/gendib.c [iso-8859-1] Sun Mar 4 13:06:02 2012 @@ -28,49 +28,49 @@ * Basically, what happens is that generic code which looks like: * * for (...) - * { + * { * if (CondA) - * { + * { * doSomethingA; - * } + * } * for (...) - * { + * { * if (CondB) - * { + * { * doSomethingB; - * } + * } * else - * { + * { * doSomethingElseB; - * } - * } - * } + * } + * } + * } * * is specialized for named rops to look like: * * if (condC) - * { + * { * if (condD) - * { + * { * for (...) - * { + * { * for (...) - * { + * { * pumpSomeBytes; - * } - * } - * } + * } + * } + * } * else - * { + * { * for (...) - * { + * { * for (...) - * { + * { * pumpSomeBytesSlightlyDifferentE; - * } - * } - * } - * } + * } + * } + * } + * } * * i.e. we make the inner loops as tight as possible. * Another optimization is to try to load/store 32 alligned bits at a time from @@ -117,14 +117,14 @@ #define ROPCODE_GENERIC 256 /* Special case */ typedef struct _ROPINFO - { +{ unsigned RopCode; const char *Name; const char *Operation; int UsesDest; int UsesSource; int UsesPattern; - } +} ROPINFO, *PROPINFO; #define FLAG_PATTERNSURFACE 0x01 @@ -136,170 +136,170 @@ static PROPINFO FindRopInfo(unsigned RopCode) { - static ROPINFO KnownCodes[] = - { - { ROPCODE_BLACKNESS, "BLACKNESS", "0", 0, 0, 0 }, - { ROPCODE_NOTSRCERASE, "NOTSRCERASE","~(D | S)", 1, 1, 0 }, - { ROPCODE_NOTSRCCOPY, "NOTSRCCOPY", "~S", 0, 1, 0 }, - { ROPCODE_SRCERASE, "SRCERASE", "(~D) & S", 1, 1, 0 }, - { ROPCODE_DSTINVERT, "DSTINVERT", "~D", 1, 0, 0 }, - { ROPCODE_PATINVERT, "PATINVERT", "D ^ P", 1, 0, 1 }, - { ROPCODE_SRCINVERT, "SRCINVERT", "D ^ S", 1, 1, 0 }, - { ROPCODE_SRCAND, "SRCAND", "D & S", 1, 1, 0 }, - { ROPCODE_NOOP, "NOOP", "D", 1, 0, 0 }, - { ROPCODE_MERGEPAINT, "MERGEPAINT", "D | (~S)", 1, 1, 0 }, - { ROPCODE_MERGECOPY, "MERGECOPY", "S & P", 0, 1, 1 }, - { ROPCODE_SRCCOPY, "SRCCOPY", "S", 0, 1, 0 }, - { ROPCODE_SRCPAINT, "SRCPAINT", "D | S", 1, 1, 0 }, - { ROPCODE_PATCOPY, "PATCOPY", "P", 0, 0, 1 }, - { ROPCODE_PATPAINT, "PATPAINT", "D | (~S) | P", 1, 1, 1 }, - { ROPCODE_WHITENESS, "WHITENESS", "0xffffffff", 0, 0, 0 }, - { ROPCODE_GENERIC, NULL, NULL, 1, 1, 1 } + static ROPINFO KnownCodes[] = + { + { ROPCODE_BLACKNESS, "BLACKNESS", "0", 0, 0, 0 }, + { ROPCODE_NOTSRCERASE, "NOTSRCERASE","~(D | S)", 1, 1, 0 }, + { ROPCODE_NOTSRCCOPY, "NOTSRCCOPY", "~S", 0, 1, 0 }, + { ROPCODE_SRCERASE, "SRCERASE", "(~D) & S", 1, 1, 0 }, + { ROPCODE_DSTINVERT, "DSTINVERT", "~D", 1, 0, 0 }, + { ROPCODE_PATINVERT, "PATINVERT", "D ^ P", 1, 0, 1 }, + { ROPCODE_SRCINVERT, "SRCINVERT", "D ^ S", 1, 1, 0 }, + { ROPCODE_SRCAND, "SRCAND", "D & S", 1, 1, 0 }, + { ROPCODE_NOOP, "NOOP", "D", 1, 0, 0 }, + { ROPCODE_MERGEPAINT, "MERGEPAINT", "D | (~S)", 1, 1, 0 }, + { ROPCODE_MERGECOPY, "MERGECOPY", "S & P", 0, 1, 1 }, + { ROPCODE_SRCCOPY, "SRCCOPY", "S", 0, 1, 0 }, + { ROPCODE_SRCPAINT, "SRCPAINT", "D | S", 1, 1, 0 }, + { ROPCODE_PATCOPY, "PATCOPY", "P", 0, 0, 1 }, + { ROPCODE_PATPAINT, "PATPAINT", "D | (~S) | P", 1, 1, 1 }, + { ROPCODE_WHITENESS, "WHITENESS", "0xffffffff", 0, 0, 0 }, + { ROPCODE_GENERIC, NULL, NULL, 1, 1, 1 } }; - unsigned Index; - - for (Index = 0; Index < sizeof(KnownCodes) / sizeof(KnownCodes[0]); Index++) - { - if (RopCode == KnownCodes[Index].RopCode) - { - return KnownCodes + Index; - } - } - - return NULL; + unsigned Index; + + for (Index = 0; Index < sizeof(KnownCodes) / sizeof(KnownCodes[0]); Index++) + { + if (RopCode == KnownCodes[Index].RopCode) + { + return KnownCodes + Index; + } + } + + return NULL; } static void Output(FILE *Out, const char *Fmt, ...) { - static unsigned Indent = 0; - static int AtBOL = 1; - unsigned n; - va_list Args; - - if (NULL != strchr(Fmt, '{') && 0 != Indent) - { - Indent += 2; - } - else if (NULL != strchr(Fmt, '}')) - { - Indent -= 2; - } - if (AtBOL) - { - for (n = 0; n < Indent; n++) - { - putc(' ', Out); - } - AtBOL = 0; - } - - va_start(Args, Fmt); - vfprintf(Out, Fmt, Args); - va_end(Args); - - if (NULL != strchr(Fmt, '{')) - { - Indent += 2; - } - else if (NULL != strchr(Fmt, '}') && 0 != Indent) - { - Indent -= 2; - } - AtBOL = '\n' == Fmt[strlen(Fmt) - 1]; + static unsigned Indent = 0; + static int AtBOL = 1; + unsigned n; + va_list Args; + + if (NULL != strchr(Fmt, '{') && 0 != Indent) + { + Indent += 2; + } + else if (NULL != strchr(Fmt, '}')) + { + Indent -= 2; + } + if (AtBOL) + { + for (n = 0; n < Indent; n++) + { + putc(' ', Out); + } + AtBOL = 0; + } + + va_start(Args, Fmt); + vfprintf(Out, Fmt, Args); + va_end(Args); + + if (NULL != strchr(Fmt, '{')) + { + Indent += 2; + } + else if (NULL != strchr(Fmt, '}') && 0 != Indent) + { + Indent -= 2; + } + AtBOL = '\n' == Fmt[strlen(Fmt) - 1]; } static void PrintRoutineName(FILE *Out, unsigned Bpp, PROPINFO RopInfo) { - if (NULL != RopInfo && ROPCODE_GENERIC != RopInfo->RopCode) - { - Output(Out, "DIB_%uBPP_BitBlt_%s", Bpp, RopInfo->Name); - } - else - { - Output(Out, "DIB_%uBPP_BitBlt_Generic", Bpp); + if (NULL != RopInfo && ROPCODE_GENERIC != RopInfo->RopCode) + { + Output(Out, "DIB_%uBPP_BitBlt_%s", Bpp, RopInfo->Name); + } + else + { + Output(Out, "DIB_%uBPP_BitBlt_Generic", Bpp); } } static void CreateShiftTables(FILE *Out) { - Output(Out, "\n"); - Output(Out, "static unsigned Shift1Bpp[] =\n"); - Output(Out, "{\n"); - Output(Out, "0,\n"); - Output(Out, "24, 25, 26, 27, 28, 29, 30, 31, 16, 17, 18, 19, 20, 21, 22, 23,\n"); - Output(Out, "8, 9, 10, 11, 12, 13, 14, 15, 0, 1, 2, 3, 4, 5, 6, 7\n"); - Output(Out, "};\n"); - Output(Out, "static unsigned Shift4Bpp[] =\n"); - Output(Out, "{\n"); - Output(Out, "0,\n"); - Output(Out, "24, 28, 16, 20, 8, 12, 0, 4\n"); - Output(Out, "};\n"); - Output(Out, "static unsigned Shift8Bpp[] =\n"); - Output(Out, "{\n"); - Output(Out, "0,\n"); - Output(Out, "24, 16, 8, 0\n"); - Output(Out, "};\n"); - Output(Out, "static unsigned Shift16Bpp[] =\n"); - Output(Out, "{\n"); - Output(Out, "0,\n"); - Output(Out, "16, 0\n"); - Output(Out, "};\n"); + Output(Out, "\n"); + Output(Out, "static unsigned Shift1Bpp[] =\n"); + Output(Out, "{\n"); + Output(Out, "0,\n"); + Output(Out, "24, 25, 26, 27, 28, 29, 30, 31, 16, 17, 18, 19, 20, 21, 22, 23,\n"); + Output(Out, "8, 9, 10, 11, 12, 13, 14, 15, 0, 1, 2, 3, 4, 5, 6, 7\n"); + Output(Out, "};\n"); + Output(Out, "static unsigned Shift4Bpp[] =\n"); + Output(Out, "{\n"); + Output(Out, "0,\n"); + Output(Out, "24, 28, 16, 20, 8, 12, 0, 4\n"); + Output(Out, "};\n"); + Output(Out, "static unsigned Shift8Bpp[] =\n"); + Output(Out, "{\n"); + Output(Out, "0,\n"); + Output(Out, "24, 16, 8, 0\n"); + Output(Out, "};\n"); + Output(Out, "static unsigned Shift16Bpp[] =\n"); + Output(Out, "{\n"); + Output(Out, "0,\n"); + Output(Out, "16, 0\n"); + Output(Out, "};\n"); } static void CreateOperation(FILE *Out, unsigned Bpp, PROPINFO RopInfo, unsigned SourceBpp, unsigned Bits) { - const char *Cast; - const char *Dest; - const char *Template; - - MARK(Out); - if (32 == Bits) - { - Cast = ""; - Dest = "*DestPtr"; - } - else if (16 == Bpp) - { - Cast = "(USHORT) "; - Dest = "*((PUSHORT) DestPtr)"; - } - else - { - Cast = "(UCHAR) "; - Dest = "*((PUCHAR) DestPtr)"; - } - Output(Out, "%s = ", Dest); - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "%sDIB_DoRop(BltInfo->Rop4, %s, Source, Pattern)", - Cast, Dest); - } - else - { - Template = RopInfo->Operation; - while ('\0' != *Template) - { - switch(*Template) + const char *Cast; + const char *Dest; + const char *Template; + + MARK(Out); + if (32 == Bits) + { + Cast = ""; + Dest = "*DestPtr"; + } + else if (16 == Bpp) + { + Cast = "(USHORT) "; + Dest = "*((PUSHORT) DestPtr)"; + } + else + { + Cast = "(UCHAR) "; + Dest = "*((PUCHAR) DestPtr)"; + } + Output(Out, "%s = ", Dest); + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "%sDIB_DoRop(BltInfo->Rop4, %s, Source, Pattern)", + Cast, Dest); + } + else + { + Template = RopInfo->Operation; + while ('\0' != *Template) + { + switch(*Template) { case 'S': - Output(Out, "%sSource", Cast); - break; + Output(Out, "%sSource", Cast); + break; case 'P': - Output(Out, "%sPattern", Cast); - break; + Output(Out, "%sPattern", Cast); + break; case 'D': - Output(Out, "%s", Dest); - break; + Output(Out, "%s", Dest); + break; default: - Output(Out, "%c", *Template); - break; - } - Template++; + Output(Out, "%c", *Template); + break; + } + Template++; } } } @@ -307,62 +307,62 @@ static void CreateBase(FILE *Out, int Source, int Flags, unsigned Bpp) { - const char *What = (Source ? "Source" : "Dest"); - - MARK(Out); - Output(Out, "%sBase = (char *) BltInfo->%sSurface->pvScan0 +\n", What, What); - if (0 == (Flags & FLAG_BOTTOMUP)) - { - if (Source) - { - Output(Out, " BltInfo->SourcePoint.y *\n"); - } - else - { - Output(Out, " BltInfo->DestRect.top *\n"); - } - } - else - { - if (Source) - { - Output(Out, " (BltInfo->SourcePoint.y +\n"); - Output(Out, " BltInfo->DestRect.bottom -\n"); - Output(Out, " BltInfo->DestRect.top - 1) *\n"); - } - else - { - Output(Out, " (BltInfo->DestRect.bottom - 1) *\n"); - } - } - Output(Out, " %sBltInfo->%sSurface->lDelta +\n", - Source ? " " : "", What); - if (Source) - { - Output(Out, " %sBltInfo->SourcePoint.x", - 16 < Bpp ? "" : "(("); - } - else - { - Output(Out, " BltInfo->DestRect.left"); - } - if (Bpp < 8) - { - Output(Out, " / %u", 8 / Bpp); - } - else if (8 < Bpp) - { - Output(Out, " * %u", Bpp / 8); - } - if (Source && Bpp <= 16) - { - Output(Out, ") & ~ 0x3)"); - } - Output(Out, ";\n", Bpp / 8); - if (Source && Bpp <= 16) - { - Output(Out, "BaseSourcePixels = %u - (BltInfo->SourcePoint.x & 0x%x);\n", - 32 / Bpp, 32 / Bpp - 1); + const char *What = (Source ? "Source" : "Dest"); + + MARK(Out); + Output(Out, "%sBase = (char *) BltInfo->%sSurface->pvScan0 +\n", What, What); + if (0 == (Flags & FLAG_BOTTOMUP)) + { + if (Source) + { + Output(Out, " BltInfo->SourcePoint.y *\n"); + } + else + { + Output(Out, " BltInfo->DestRect.top *\n"); + } + } + else + { + if (Source) + { + Output(Out, " (BltInfo->SourcePoint.y +\n"); + Output(Out, " BltInfo->DestRect.bottom -\n"); + Output(Out, " BltInfo->DestRect.top - 1) *\n"); + } + else + { + Output(Out, " (BltInfo->DestRect.bottom - 1) *\n"); + } + } + Output(Out, " %sBltInfo->%sSurface->lDelta +\n", + Source ? " " : "", What); + if (Source) + { + Output(Out, " %sBltInfo->SourcePoint.x", + 16 < Bpp ? "" : "(("); + } + else + { + Output(Out, " BltInfo->DestRect.left"); + } + if (Bpp < 8) + { + Output(Out, " / %u", 8 / Bpp); + } + else if (8 < Bpp) + { + Output(Out, " * %u", Bpp / 8); + } + if (Source && Bpp <= 16) + { + Output(Out, ") & ~ 0x3)"); + } + Output(Out, ";\n", Bpp / 8); + if (Source && Bpp <= 16) + { + Output(Out, "BaseSourcePixels = %u - (BltInfo->SourcePoint.x & 0x%x);\n", + 32 / Bpp, 32 / Bpp - 1); } } @@ -370,79 +370,79 @@ CreateGetSource(FILE *Out, unsigned Bpp, PROPINFO RopInfo, int Flags, unsigned SourceBpp, unsigned Shift) { - const char *AssignOp; - const char *Before; - char After[8]; - - MARK(Out); - if (0 == Shift) - { - AssignOp = "="; - Before = ""; - After[0] = '\0'; - } - else - { - AssignOp = "|="; - Before = "("; - sprintf(After, ") << %u", Shift); - } - - if (ROPCODE_SRCCOPY != RopInfo->RopCode || - 0 == (Flags & FLAG_TRIVIALXLATE) || Bpp != SourceBpp) - { - if (0 == (Flags & FLAG_FORCERAWSOURCEAVAIL) && SourceBpp <= 16) - { - Output(Out, "if (0 == SourcePixels)\n"); - Output(Out, "{\n"); - Output(Out, "RawSource = *SourcePtr++;\n"); - Output(Out, "SourcePixels = %u;\n", 32 / SourceBpp); - Output(Out, "}\n"); - } - Output(Out, "Source %s (%s", AssignOp, Before); - if (0 == (Flags & FLAG_TRIVIALXLATE)) - { - Output(Out, "XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, %s", - 16 < SourceBpp ? "" : "("); - } - if (32 == SourceBpp) - { - Output(Out, "*SourcePtr++"); - } - else if (24 == SourceBpp) - { - Output(Out, "*(PUSHORT) SourcePtr + (*((PBYTE) SourcePtr + 2) << 16)"); - } - else - { - Output(Out, "RawSource >> Shift%uBpp[SourcePixels]", SourceBpp); - } - if (0 == (Flags & FLAG_TRIVIALXLATE)) - { - if (16 < SourceBpp) - { - Output(Out, ")"); - } - else - { - Output(Out, ") & 0x%x)", (1 << SourceBpp) - 1); - } - } - if (32 == Bpp) - { - Output(Out, ")%s;\n", After); - } - else - { - Output(Out, " & 0x%x)%s;\n", (1 << Bpp) - 1, After); - } - if (SourceBpp <= 16) - { - Output(Out, "SourcePixels--;\n"); - } - else if (24 == SourceBpp) - { - Output(Out, "SourcePtr = (PULONG)((char *) SourcePtr + 3);\n"); + const char *AssignOp; + const char *Before; + char After[8]; + + MARK(Out); + if (0 == Shift) + { + AssignOp = "="; + Before = ""; + After[0] = '\0'; + } + else + { + AssignOp = "|="; + Before = "("; + sprintf(After, ") << %u", Shift); + } + + if (ROPCODE_SRCCOPY != RopInfo->RopCode || + 0 == (Flags & FLAG_TRIVIALXLATE) || Bpp != SourceBpp) + { + if (0 == (Flags & FLAG_FORCERAWSOURCEAVAIL) && SourceBpp <= 16) + { + Output(Out, "if (0 == SourcePixels)\n"); + Output(Out, "{\n"); + Output(Out, "RawSource = *SourcePtr++;\n"); + Output(Out, "SourcePixels = %u;\n", 32 / SourceBpp); + Output(Out, "}\n"); + } + Output(Out, "Source %s (%s", AssignOp, Before); + if (0 == (Flags & FLAG_TRIVIALXLATE)) + { + Output(Out, "XLATEOBJ_iXlate(BltInfo->XlateSourceToDest, %s", + 16 < SourceBpp ? "" : "("); + } + if (32 == SourceBpp) + { + Output(Out, "*SourcePtr++"); + } + else if (24 == SourceBpp) + { + Output(Out, "*(PUSHORT) SourcePtr + (*((PBYTE) SourcePtr + 2) << 16)"); + } + else + { + Output(Out, "RawSource >> Shift%uBpp[SourcePixels]", SourceBpp); + } + if (0 == (Flags & FLAG_TRIVIALXLATE)) + { + if (16 < SourceBpp) + { + Output(Out, ")"); + } + else + { + Output(Out, ") & 0x%x)", (1 << SourceBpp) - 1); + } + } + if (32 == Bpp) + { + Output(Out, ")%s;\n", After); + } + else + { + Output(Out, " & 0x%x)%s;\n", (1 << Bpp) - 1, After); + } + if (SourceBpp <= 16) + { + Output(Out, "SourcePixels--;\n"); + } + else if (24 == SourceBpp) + { + Output(Out, "SourcePtr = (PULONG)((char *) SourcePtr + 3);\n"); } } } @@ -450,31 +450,31 @@ static void CreateCounts(FILE *Out, unsigned Bpp) { - MARK(Out); - if (32 != Bpp) - { - if (8 < Bpp) - { - Output(Out, "LeftCount = ((ULONG_PTR) DestBase >> 1) & 0x01;\n"); - } - else - { - Output(Out, "LeftCount = (ULONG_PTR) DestBase & 0x03;\n"); - Output(Out, "if (BltInfo->DestRect.right - BltInfo->DestRect.left < " - "LeftCount)\n"); - Output(Out, "{\n"); - Output(Out, "LeftCount = BltInfo->DestRect.right - " - "BltInfo->DestRect.left;\n"); - Output(Out, "}\n"); - } - Output(Out, "CenterCount = (BltInfo->DestRect.right - BltInfo->DestRect.left -\n"); - Output(Out, " LeftCount) / %u;\n", 32 / Bpp); - Output(Out, "RightCount = (BltInfo->DestRect.right - BltInfo->DestRect.left -\n"); - Output(Out, " LeftCount - %u * CenterCount);\n", 32 / Bpp); - } - else - { - Output(Out, "CenterCount = BltInfo->DestRect.right - BltInfo->DestRect.left;\n"); + MARK(Out); + if (32 != Bpp) + { + if (8 < Bpp) + { + Output(Out, "LeftCount = ((ULONG_PTR) DestBase >> 1) & 0x01;\n"); + } + else + { + Output(Out, "LeftCount = (ULONG_PTR) DestBase & 0x03;\n"); + Output(Out, "if (BltInfo->DestRect.right - BltInfo->DestRect.left < " + "LeftCount)\n"); + Output(Out, "{\n"); + Output(Out, "LeftCount = BltInfo->DestRect.right - " + "BltInfo->DestRect.left;\n"); + Output(Out, "}\n"); + } + Output(Out, "CenterCount = (BltInfo->DestRect.right - BltInfo->DestRect.left -\n"); + Output(Out, " LeftCount) / %u;\n", 32 / Bpp); + Output(Out, "RightCount = (BltInfo->DestRect.right - BltInfo->DestRect.left -\n"); + Output(Out, " LeftCount - %u * CenterCount);\n", 32 / Bpp); + } + else + { + Output(Out, "CenterCount = BltInfo->DestRect.right - BltInfo->DestRect.left;\n"); } } @@ -482,562 +482,562 @@ CreateSetSinglePixel(FILE *Out, unsigned Bpp, PROPINFO RopInfo, int Flags, unsigned SourceBpp) { - if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) - { - CreateGetSource(Out, Bpp, RopInfo, Flags, SourceBpp, 0); - MARK(Out); - } - if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) - { - Output(Out, "Pattern = DIB_GetSourceIndex(BltInfo->PatternSurface, PatternX, PatternY);\n"); - Output(Out, "if (BltInfo->PatternSurface->sizlBitmap.cx <= ++PatternX)\n"); - Output(Out, "{\n"); - Output(Out, "PatternX -= BltInfo->PatternSurface->sizlBitmap.cx;\n"); - Output(Out, "}\n"); - } - if ((RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE) && - Bpp != SourceBpp) || - (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE))) - { - Output(Out, "\n"); - } - CreateOperation(Out, Bpp, RopInfo, SourceBpp, 16); - Output(Out, ";\n"); - MARK(Out); - Output(Out, "\n"); - Output(Out, "DestPtr = (PULONG)((char *) DestPtr + %u);\n", Bpp / 8); + if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) + { + CreateGetSource(Out, Bpp, RopInfo, Flags, SourceBpp, 0); + MARK(Out); + } + if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) + { + Output(Out, "Pattern = DIB_GetSourceIndex(BltInfo->PatternSurface, PatternX, PatternY);\n"); + Output(Out, "if (BltInfo->PatternSurface->sizlBitmap.cx <= ++PatternX)\n"); + Output(Out, "{\n"); + Output(Out, "PatternX -= BltInfo->PatternSurface->sizlBitmap.cx;\n"); + Output(Out, "}\n"); + } + if ((RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE) && + Bpp != SourceBpp) || + (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE))) + { + Output(Out, "\n"); + } + CreateOperation(Out, Bpp, RopInfo, SourceBpp, 16); + Output(Out, ";\n"); + MARK(Out); + Output(Out, "\n"); + Output(Out, "DestPtr = (PULONG)((char *) DestPtr + %u);\n", Bpp / 8); } static void CreateBitCase(FILE *Out, unsigned Bpp, PROPINFO RopInfo, int Flags, unsigned SourceBpp) { - unsigned Partial; - - MARK(Out); - if (RopInfo->UsesSource) - { - if (0 == (Flags & FLAG_FORCENOUSESSOURCE)) - { - CreateBase(Out, 1, Flags, SourceBpp); - } - CreateBase(Out, 0, Flags, Bpp); - CreateCounts(Out, Bpp); - MARK(Out); - } - if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) - { - if (0 == (Flags & FLAG_BOTTOMUP)) - { - Output(Out, "PatternY = (BltInfo->DestRect.top - BltInfo->BrushOrigin.y) %%\n"); - Output(Out, " BltInfo->PatternSurface->sizlBitmap.cy;\n"); - } - else - { - Output(Out, "PatternY = (BltInfo->DestRect.bottom - 1 -\n"); - Output(Out, " BltInfo->BrushOrigin.y) %%\n"); - Output(Out, " BltInfo->PatternSurface->sizlBitmap.cy;\n"); - } - } - if (ROPCODE_SRCCOPY == RopInfo->RopCode && - 0 != (Flags & FLAG_TRIVIALXLATE) && Bpp == SourceBpp) - { - Output(Out, "CenterCount = %u * (BltInfo->DestRect.right -\n", Bpp >> 3); - Output(Out, " BltInfo->DestRect.left);\n"); - } - if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) - { - Output(Out, "BasePatternX = (BltInfo->DestRect.left - BltInfo->BrushOrigin.x) %%\n"); - Output(Out, " BltInfo->PatternSurface->sizlBitmap.cx;\n"); - } - - Output(Out, "for (LineIndex = 0; LineIndex < LineCount; LineIndex++)\n"); - Output(Out, "{\n"); - if (ROPCODE_SRCCOPY != RopInfo->RopCode || - 0 == (Flags & FLAG_TRIVIALXLATE) || Bpp != SourceBpp) - { - if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) - { - Output(Out, "SourcePtr = (PULONG) SourceBase;\n"); - if (SourceBpp <= 16) - { - Output(Out, "RawSource = *SourcePtr++;\n"); - Output(Out, "SourcePixels = BaseSourcePixels;\n"); - } - } - Output(Out, "DestPtr = (PULONG) DestBase;\n"); - } - - if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) - { - Output(Out, "PatternX = BasePatternX;\n"); - } - - if (ROPCODE_SRCCOPY == RopInfo->RopCode && - 0 != (Flags & FLAG_TRIVIALXLATE) && Bpp == SourceBpp) - { - Output(Out, "RtlMoveMemory(DestBase, SourceBase, CenterCount);\n"); - Output(Out, "\n"); - } - else - { - Output(Out, "\n"); - if (32 != Bpp) - { - if (16 == Bpp) - { - Output(Out, "if (0 != LeftCount)\n"); - } - else - { - Output(Out, "for (i = 0; i < LeftCount; i++)\n"); - } - Output(Out, "{\n"); - CreateSetSinglePixel(Out, Bpp, RopInfo, - (16 == Bpp ? Flags | FLAG_FORCERAWSOURCEAVAIL : - Flags), SourceBpp); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "\n"); - } - Output(Out, "for (i = 0; i < CenterCount; i++)\n"); - Output(Out, "{\n"); - if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) - { - for (Partial = 0; Partial < 32 / Bpp; Partial++) - { - CreateGetSource(Out, Bpp, RopInfo, Flags, SourceBpp, - Partial * Bpp); - MARK(Out); - } - Output(Out, "\n"); - } - if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) - { - for (Partial = 0; Partial < 32 / Bpp; Partial++) - { - if (0 == Partial) + unsigned Partial; + + MARK(Out); + if (RopInfo->UsesSource) + { + if (0 == (Flags & FLAG_FORCENOUSESSOURCE)) + { + CreateBase(Out, 1, Flags, SourceBpp); + } + CreateBase(Out, 0, Flags, Bpp); + CreateCounts(Out, Bpp); + MARK(Out); + } + if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) + { + if (0 == (Flags & FLAG_BOTTOMUP)) + { + Output(Out, "PatternY = (BltInfo->DestRect.top - BltInfo->BrushOrigin.y) %%\n"); + Output(Out, " BltInfo->PatternSurface->sizlBitmap.cy;\n"); + } + else + { + Output(Out, "PatternY = (BltInfo->DestRect.bottom - 1 -\n"); + Output(Out, " BltInfo->BrushOrigin.y) %%\n"); + Output(Out, " BltInfo->PatternSurface->sizlBitmap.cy;\n"); + } + } + if (ROPCODE_SRCCOPY == RopInfo->RopCode && + 0 != (Flags & FLAG_TRIVIALXLATE) && Bpp == SourceBpp) + { + Output(Out, "CenterCount = %u * (BltInfo->DestRect.right -\n", Bpp >> 3); + Output(Out, " BltInfo->DestRect.left);\n"); + } + if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) + { + Output(Out, "BasePatternX = (BltInfo->DestRect.left - BltInfo->BrushOrigin.x) %%\n"); + Output(Out, " BltInfo->PatternSurface->sizlBitmap.cx;\n"); + } + + Output(Out, "for (LineIndex = 0; LineIndex < LineCount; LineIndex++)\n"); + Output(Out, "{\n"); + if (ROPCODE_SRCCOPY != RopInfo->RopCode || + 0 == (Flags & FLAG_TRIVIALXLATE) || Bpp != SourceBpp) + { + if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) + { + Output(Out, "SourcePtr = (PULONG) SourceBase;\n"); + if (SourceBpp <= 16) + { + Output(Out, "RawSource = *SourcePtr++;\n"); + Output(Out, "SourcePixels = BaseSourcePixels;\n"); + } + } + Output(Out, "DestPtr = (PULONG) DestBase;\n"); + } + + if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) + { + Output(Out, "PatternX = BasePatternX;\n"); + } + + if (ROPCODE_SRCCOPY == RopInfo->RopCode && + 0 != (Flags & FLAG_TRIVIALXLATE) && Bpp == SourceBpp) + { + Output(Out, "RtlMoveMemory(DestBase, SourceBase, CenterCount);\n"); + Output(Out, "\n"); + } + else + { + Output(Out, "\n"); + if (32 != Bpp) + { + if (16 == Bpp) + { + Output(Out, "if (0 != LeftCount)\n"); + } + else + { + Output(Out, "for (i = 0; i < LeftCount; i++)\n"); + } + Output(Out, "{\n"); + CreateSetSinglePixel(Out, Bpp, RopInfo, + (16 == Bpp ? Flags | FLAG_FORCERAWSOURCEAVAIL : + Flags), SourceBpp); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "\n"); + } + Output(Out, "for (i = 0; i < CenterCount; i++)\n"); + Output(Out, "{\n"); + if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) + { + for (Partial = 0; Partial < 32 / Bpp; Partial++) + { + CreateGetSource(Out, Bpp, RopInfo, Flags, SourceBpp, + Partial * Bpp); + MARK(Out); + } + Output(Out, "\n"); + } + if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) + { + for (Partial = 0; Partial < 32 / Bpp; Partial++) + { + if (0 == Partial) { - Output(Out, "Pattern = DIB_GetSourceIndex(BltInfo->PatternSurface, PatternX, PatternY);\n"); + Output(Out, "Pattern = DIB_GetSourceIndex(BltInfo->PatternSurface, PatternX, PatternY);\n"); } - else + else { - Output(Out, "Pattern |= DIB_GetSourceIndex(BltInfo->PatternSurface, PatternX, PatternY) << %u;\n", Partial * Bpp); + Output(Out, "Pattern |= DIB_GetSourceIndex(BltInfo->PatternSurface, PatternX, PatternY) << %u;\n", Partial * Bpp); } - Output(Out, "if (BltInfo->PatternSurface->sizlBitmap.cx <= ++PatternX)\n"); - Output(Out, "{\n"); - Output(Out, "PatternX -= BltInfo->PatternSurface->sizlBitmap.cx;\n"); - Output(Out, "}\n"); - } - Output(Out, "\n"); - } - CreateOperation(Out, Bpp, RopInfo, SourceBpp, 32); - Output(Out, ";\n"); - MARK(Out); - Output(Out, "\n"); - Output(Out, "DestPtr++;\n"); - Output(Out, "}\n"); - Output(Out, "\n"); - if (32 != Bpp) - { - if (16 == Bpp) - { - Output(Out, "if (0 != RightCount)\n"); - } - else - { - Output(Out, "for (i = 0; i < RightCount; i++)\n"); - } - Output(Out, "{\n"); - CreateSetSinglePixel(Out, Bpp, RopInfo, Flags, SourceBpp); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "\n"); - } - if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) - { - if (0 == (Flags & FLAG_BOTTOMUP)) - { - Output(Out, "if (BltInfo->PatternSurface->sizlBitmap.cy <= ++PatternY)\n"); - Output(Out, "{\n"); - Output(Out, "PatternY -= BltInfo->PatternSurface->sizlBitmap.cy;\n"); - Output(Out, "}\n"); - } - else - { - Output(Out, "if (0 == PatternY--)\n"); - Output(Out, "{\n"); - Output(Out, "PatternY = BltInfo->PatternSurface->sizlBitmap.cy - 1;\n"); - Output(Out, "}\n"); - } - } - } - if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) - { - Output(Out, "SourceBase %c= BltInfo->SourceSurface->lDelta;\n", - 0 == (Flags & FLAG_BOTTOMUP) ? '+' : '-'); - } - Output(Out, "DestBase %c= BltInfo->DestSurface->lDelta;\n", - 0 == (Flags & FLAG_BOTTOMUP) ? '+' : '-'); - Output(Out, "}\n"); + Output(Out, "if (BltInfo->PatternSurface->sizlBitmap.cx <= ++PatternX)\n"); + Output(Out, "{\n"); + Output(Out, "PatternX -= BltInfo->PatternSurface->sizlBitmap.cx;\n"); + Output(Out, "}\n"); + } + Output(Out, "\n"); + } + CreateOperation(Out, Bpp, RopInfo, SourceBpp, 32); + Output(Out, ";\n"); + MARK(Out); + Output(Out, "\n"); + Output(Out, "DestPtr++;\n"); + Output(Out, "}\n"); + Output(Out, "\n"); + if (32 != Bpp) + { + if (16 == Bpp) + { + Output(Out, "if (0 != RightCount)\n"); + } + else + { + Output(Out, "for (i = 0; i < RightCount; i++)\n"); + } + Output(Out, "{\n"); + CreateSetSinglePixel(Out, Bpp, RopInfo, Flags, SourceBpp); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "\n"); + } + if (RopInfo->UsesPattern && 0 != (Flags & FLAG_PATTERNSURFACE)) + { + if (0 == (Flags & FLAG_BOTTOMUP)) + { + Output(Out, "if (BltInfo->PatternSurface->sizlBitmap.cy <= ++PatternY)\n"); + Output(Out, "{\n"); + Output(Out, "PatternY -= BltInfo->PatternSurface->sizlBitmap.cy;\n"); + Output(Out, "}\n"); + } + else + { + Output(Out, "if (0 == PatternY--)\n"); + Output(Out, "{\n"); + Output(Out, "PatternY = BltInfo->PatternSurface->sizlBitmap.cy - 1;\n"); + Output(Out, "}\n"); + } + } + } + if (RopInfo->UsesSource && 0 == (Flags & FLAG_FORCENOUSESSOURCE)) + { + Output(Out, "SourceBase %c= BltInfo->SourceSurface->lDelta;\n", + 0 == (Flags & FLAG_BOTTOMUP) ? '+' : '-'); + } + Output(Out, "DestBase %c= BltInfo->DestSurface->lDelta;\n", + 0 == (Flags & FLAG_BOTTOMUP) ? '+' : '-'); + Output(Out, "}\n"); } static void CreateActionBlock(FILE *Out, unsigned Bpp, PROPINFO RopInfo, int Flags) { - static unsigned SourceBpp[ ] = + static unsigned SourceBpp[ ] = { 1, 4, 8, 16, 24, 32 }; - unsigned BppIndex; - - MARK(Out); - if (RopInfo->UsesSource) - { - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "if (UsesSource)\n"); - Output(Out, "{\n"); - } - Output(Out, "switch (BltInfo->SourceSurface->iBitmapFormat)\n"); - Output(Out, "{\n"); - for (BppIndex = 0; - BppIndex < sizeof(SourceBpp) / sizeof(unsigned); - BppIndex++) - { - Output(Out, "case BMF_%uBPP:\n", SourceBpp[BppIndex]); - Output(Out, "{\n"); - if (Bpp == SourceBpp[BppIndex]) - { - Output(Out, "if (NULL == BltInfo->XlateSourceToDest ||\n"); - Output(Out, " 0 != (BltInfo->XlateSourceToDest->flXlate & XO_TRIVIAL))\n"); - Output(Out, "{\n"); - Output(Out, "if (BltInfo->DestRect.top < BltInfo->SourcePoint.y)\n"); - Output(Out, "{\n"); - CreateBitCase(Out, Bpp, RopInfo, - Flags | FLAG_TRIVIALXLATE, - SourceBpp[BppIndex]); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "else\n"); - Output(Out, "{\n"); - CreateBitCase(Out, Bpp, RopInfo, - Flags | FLAG_BOTTOMUP | FLAG_TRIVIALXLATE, - SourceBpp[BppIndex]); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "}\n"); - Output(Out, "else\n"); - Output(Out, "{\n"); - Output(Out, "if (BltInfo->DestRect.top < BltInfo->SourcePoint.y)\n"); - Output(Out, "{\n"); - CreateBitCase(Out, Bpp, RopInfo, Flags, SourceBpp[BppIndex]); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "else\n"); - Output(Out, "{\n"); - CreateBitCase(Out, Bpp, RopInfo, - Flags | FLAG_BOTTOMUP, - SourceBpp[BppIndex]); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "}\n"); - } - else - { - CreateBitCase(Out, Bpp, RopInfo, Flags, - SourceBpp[BppIndex]); - MARK(Out); - } - Output(Out, "break;\n"); - Output(Out, "}\n"); - } - Output(Out, "}\n"); - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "}\n"); - Output(Out, "else\n"); - Output(Out, "{\n"); - CreateBitCase(Out, Bpp, RopInfo, Flags | FLAG_FORCENOUSESSOURCE, 0); - MARK(Out); - Output(Out, "}\n"); - } - } - else - { - CreateBitCase(Out, Bpp, RopInfo, Flags, 0); + unsigned BppIndex; + + MARK(Out); + if (RopInfo->UsesSource) + { + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "if (UsesSource)\n"); + Output(Out, "{\n"); + } + Output(Out, "switch (BltInfo->SourceSurface->iBitmapFormat)\n"); + Output(Out, "{\n"); + for (BppIndex = 0; + BppIndex < sizeof(SourceBpp) / sizeof(unsigned); + BppIndex++) + { + Output(Out, "case BMF_%uBPP:\n", SourceBpp[BppIndex]); + Output(Out, "{\n"); + if (Bpp == SourceBpp[BppIndex]) + { + Output(Out, "if (NULL == BltInfo->XlateSourceToDest ||\n"); + Output(Out, " 0 != (BltInfo->XlateSourceToDest->flXlate & XO_TRIVIAL))\n"); + Output(Out, "{\n"); + Output(Out, "if (BltInfo->DestRect.top < BltInfo->SourcePoint.y)\n"); + Output(Out, "{\n"); + CreateBitCase(Out, Bpp, RopInfo, + Flags | FLAG_TRIVIALXLATE, + SourceBpp[BppIndex]); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "else\n"); + Output(Out, "{\n"); + CreateBitCase(Out, Bpp, RopInfo, + Flags | FLAG_BOTTOMUP | FLAG_TRIVIALXLATE, + SourceBpp[BppIndex]); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "}\n"); + Output(Out, "else\n"); + Output(Out, "{\n"); + Output(Out, "if (BltInfo->DestRect.top < BltInfo->SourcePoint.y)\n"); + Output(Out, "{\n"); + CreateBitCase(Out, Bpp, RopInfo, Flags, SourceBpp[BppIndex]); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "else\n"); + Output(Out, "{\n"); + CreateBitCase(Out, Bpp, RopInfo, + Flags | FLAG_BOTTOMUP, + SourceBpp[BppIndex]); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "}\n"); + } + else + { + CreateBitCase(Out, Bpp, RopInfo, Flags, + SourceBpp[BppIndex]); + MARK(Out); + } + Output(Out, "break;\n"); + Output(Out, "}\n"); + } + Output(Out, "}\n"); + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "}\n"); + Output(Out, "else\n"); + Output(Out, "{\n"); + CreateBitCase(Out, Bpp, RopInfo, Flags | FLAG_FORCENOUSESSOURCE, 0); + MARK(Out); + Output(Out, "}\n"); + } + } + else + { + CreateBitCase(Out, Bpp, RopInfo, Flags, 0); } } static void CreatePrimitive(FILE *Out, unsigned Bpp, PROPINFO RopInfo) { - int First; - unsigned Partial; - - MARK(Out); - Output(Out, "\n"); - Output(Out, "static void\n"); - PrintRoutineName(Out, Bpp, RopInfo); - Output(Out, "(PBLTINFO BltInfo)\n"); - Output(Out, "{\n"); - if (ROPCODE_BLACKNESS == RopInfo->RopCode) - { - Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " - "&BltInfo->DestRect, 0x0);\n", Bpp); - } - else if (ROPCODE_WHITENESS == RopInfo->RopCode) - { - Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " - "&BltInfo->DestRect, ~0);\n", Bpp); - } - else if (ROPCODE_NOOP == RopInfo->RopCode) - { - Output(Out, "return;\n"); - } - else - { - Output(Out, "ULONG LineIndex, LineCount;\n"); - Output(Out, "ULONG i;\n"); - if (RopInfo->UsesPattern) - { - Output(Out, "ULONG PatternX =0, PatternY = 0, BasePatternX = 0;\n"); - } - First = 1; - if (RopInfo->UsesSource) - { - Output(Out, "ULONG Source = 0"); - First = 0; - } - if (RopInfo->UsesPattern) - { - Output(Out, "%s Pattern = 0", First ? "ULONG" : ","); - First = 0; - } - if (! First) - { - Output(Out, ";\n"); - } - Output(Out, "char *DestBase;\n"); - Output(Out, "PULONG DestPtr;\n"); - if (RopInfo->UsesSource) - { - Output(Out, "char *SourceBase;\n"); - Output(Out, "PULONG SourcePtr;\n"); - Output(Out, "ULONG RawSource;\n"); - Output(Out, "unsigned SourcePixels, BaseSourcePixels;\n"); - } - if (32 == Bpp) - { - Output(Out, "ULONG CenterCount;\n"); - } - else - { - Output(Out, "ULONG LeftCount, CenterCount, RightCount;\n"); - } - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "BOOLEAN UsesSource, UsesPattern;\n"); - Output(Out, "\n"); - Output(Out, "UsesSource = ROP4_USES_SOURCE(BltInfo->Rop4);\n"); - Output(Out, "UsesPattern = ROP4_USES_PATTERN(BltInfo->Rop4);\n"); - } - Output(Out, "\n"); - if (! RopInfo->UsesSource) - { - CreateBase(Out, 0, 0, Bpp); - CreateCounts(Out, Bpp); - MARK(Out); - } - Output(Out, "LineCount = BltInfo->DestRect.bottom - BltInfo->DestRect.top;\n"); - - Output(Out, "\n"); - if (RopInfo->UsesPattern) - { - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "if (UsesPattern && NULL != BltInfo->PatternSurface)\n"); - } - else - { - Output(Out, "if (NULL != BltInfo->PatternSurface)\n"); - } - Output(Out, "{\n"); - CreateActionBlock(Out, Bpp, RopInfo, FLAG_PATTERNSURFACE); - MARK(Out); - Output(Out, "}\n"); - Output(Out, "else\n"); - Output(Out, "{\n"); - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "if (UsesPattern)\n"); - Output(Out, "{\n"); - } - for (Partial = 0; Partial < 32 / Bpp; Partial++) - { - if (0 == Partial) + int First; + unsigned Partial; + + MARK(Out); + Output(Out, "\n"); + Output(Out, "static void\n"); + PrintRoutineName(Out, Bpp, RopInfo); + Output(Out, "(PBLTINFO BltInfo)\n"); + Output(Out, "{\n"); + if (ROPCODE_BLACKNESS == RopInfo->RopCode) + { + Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " + "&BltInfo->DestRect, 0x0);\n", Bpp); + } + else if (ROPCODE_WHITENESS == RopInfo->RopCode) + { + Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " + "&BltInfo->DestRect, ~0);\n", Bpp); + } + else if (ROPCODE_NOOP == RopInfo->RopCode) + { + Output(Out, "return;\n"); + } + else + { + Output(Out, "ULONG LineIndex, LineCount;\n"); + Output(Out, "ULONG i;\n"); + if (RopInfo->UsesPattern) + { + Output(Out, "ULONG PatternX =0, PatternY = 0, BasePatternX = 0;\n"); + } + First = 1; + if (RopInfo->UsesSource) + { + Output(Out, "ULONG Source = 0"); + First = 0; + } + if (RopInfo->UsesPattern) + { + Output(Out, "%s Pattern = 0", First ? "ULONG" : ","); + First = 0; + } + if (! First) + { + Output(Out, ";\n"); + } + Output(Out, "char *DestBase;\n"); + Output(Out, "PULONG DestPtr;\n"); + if (RopInfo->UsesSource) + { + Output(Out, "char *SourceBase;\n"); + Output(Out, "PULONG SourcePtr;\n"); + Output(Out, "ULONG RawSource;\n"); + Output(Out, "unsigned SourcePixels, BaseSourcePixels;\n"); + } + if (32 == Bpp) + { + Output(Out, "ULONG CenterCount;\n"); + } + else + { + Output(Out, "ULONG LeftCount, CenterCount, RightCount;\n"); + } + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "BOOLEAN UsesSource, UsesPattern;\n"); + Output(Out, "\n"); + Output(Out, "UsesSource = ROP4_USES_SOURCE(BltInfo->Rop4);\n"); + Output(Out, "UsesPattern = ROP4_USES_PATTERN(BltInfo->Rop4);\n"); + } + Output(Out, "\n"); + if (! RopInfo->UsesSource) + { + CreateBase(Out, 0, 0, Bpp); + CreateCounts(Out, Bpp); + MARK(Out); + } + Output(Out, "LineCount = BltInfo->DestRect.bottom - BltInfo->DestRect.top;\n"); + + Output(Out, "\n"); + if (RopInfo->UsesPattern) + { + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "if (UsesPattern && NULL != BltInfo->PatternSurface)\n"); + } + else + { + Output(Out, "if (NULL != BltInfo->PatternSurface)\n"); + } + Output(Out, "{\n"); + CreateActionBlock(Out, Bpp, RopInfo, FLAG_PATTERNSURFACE); + MARK(Out); + Output(Out, "}\n"); + Output(Out, "else\n"); + Output(Out, "{\n"); + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "if (UsesPattern)\n"); + Output(Out, "{\n"); + } + for (Partial = 0; Partial < 32 / Bpp; Partial++) + { + if (0 == Partial) { - Output(Out, "if (!BltInfo->Brush)\n"); - Output(Out, "{\n"); - Output(Out, "Pattern = 0;\n"); - Output(Out, "}\n"); - Output(Out, "else\n"); - Output(Out, "{\n"); - Output(Out, "Pattern = BltInfo->Brush->iSolidColor"); + Output(Out, "if (!BltInfo->Brush)\n"); + Output(Out, "{\n"); + Output(Out, "Pattern = 0;\n"); + Output(Out, "}\n"); + Output(Out, "else\n"); + Output(Out, "{\n"); + Output(Out, "Pattern = BltInfo->Brush->iSolidColor"); } - else + else { - Output(Out, " (BltInfo->Brush->iSolidColor << %d)", - Partial * Bpp); + Output(Out, " (BltInfo->Brush->iSolidColor << %d)", + Partial * Bpp); } - if (32 / Bpp <= Partial + 1) + if (32 / Bpp <= Partial + 1) { - Output(Out, ";\n"); - Output(Out, "}\n"); + Output(Out, ";\n"); + Output(Out, "}\n"); } - else + else { - Output(Out, " |\n"); + Output(Out, " |\n"); } } - if (ROPCODE_PATINVERT == RopInfo->RopCode || - ROPCODE_MERGECOPY == RopInfo->RopCode) - { - Output(Out, "if (0 == Pattern)\n"); - Output(Out, "{\n"); - if (ROPCODE_MERGECOPY == RopInfo->RopCode) + if (ROPCODE_PATINVERT == RopInfo->RopCode || + ROPCODE_MERGECOPY == RopInfo->RopCode) + { + Output(Out, "if (0 == Pattern)\n"); + Output(Out, "{\n"); + if (ROPCODE_MERGECOPY == RopInfo->RopCode) { - Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " - "&BltInfo->DestRect, 0x0);\n", Bpp); + Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " + "&BltInfo->DestRect, 0x0);\n", Bpp); } - Output(Out, "return;\n"); - Output(Out, "}\n"); - } - else if (ROPCODE_PATPAINT == RopInfo->RopCode) - { - Output(Out, "if ((~0) == Pattern)\n"); - Output(Out, "{\n"); - Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " - "&BltInfo->DestRect, ~0);\n", Bpp); - Output(Out, "return;\n"); - Output(Out, "}\n"); - } - if (ROPCODE_GENERIC == RopInfo->RopCode) - { - Output(Out, "}\n"); - } - CreateActionBlock(Out, Bpp, RopInfo, 0); - MARK(Out); - Output(Out, "}\n"); - } - else - { - CreateActionBlock(Out, Bpp, RopInfo, 0); - MARK(Out); - } - } - Output(Out, "}\n"); + Output(Out, "return;\n"); + Output(Out, "}\n"); + } + else if (ROPCODE_PATPAINT == RopInfo->RopCode) + { + Output(Out, "if ((~0) == Pattern)\n"); + Output(Out, "{\n"); + Output(Out, "DIB_%uBPP_ColorFill(BltInfo->DestSurface, " + "&BltInfo->DestRect, ~0);\n", Bpp); + Output(Out, "return;\n"); + Output(Out, "}\n"); + } + if (ROPCODE_GENERIC == RopInfo->RopCode) + { + Output(Out, "}\n"); + } + CreateActionBlock(Out, Bpp, RopInfo, 0); + MARK(Out); + Output(Out, "}\n"); + } + else + { + CreateActionBlock(Out, Bpp, RopInfo, 0); + MARK(Out); + } + } + Output(Out, "}\n"); } static void CreateTable(FILE *Out, unsigned Bpp) { - unsigned RopCode; - - MARK(Out); - Output(Out, "\n"); - Output(Out, "static void (*PrimitivesTable[256])(PBLTINFO) =\n"); - Output(Out, "{\n"); - for (RopCode = 0; RopCode < 256; RopCode++) - { - PrintRoutineName(Out, Bpp, FindRopInfo(RopCode)); - if (RopCode < 255) - { - Output(Out, ","); - } - Output(Out, "\n"); - } - Output(Out, "};\n"); + unsigned RopCode; + + MARK(Out); + Output(Out, "\n"); + Output(Out, "static void (*PrimitivesTable[256])(PBLTINFO) =\n"); + Output(Out, "{\n"); + for (RopCode = 0; RopCode < 256; RopCode++) + { + PrintRoutineName(Out, Bpp, FindRopInfo(RopCode)); + if (RopCode < 255) + { + Output(Out, ","); + } + Output(Out, "\n"); + } + Output(Out, "};\n"); } static void CreateBitBlt(FILE *Out, unsigned Bpp) { - MARK(Out); - Output(Out, "\n"); - Output(Out, "BOOLEAN\n"); - Output(Out, "DIB_%uBPP_BitBlt(PBLTINFO BltInfo)\n", Bpp); - Output(Out, "{\n"); - Output(Out, "PrimitivesTable[BltInfo->Rop4 & 0xff](BltInfo);\n"); - Output(Out, "\n"); - Output(Out, "return TRUE;\n"); - Output(Out, "}\n"); + MARK(Out); + Output(Out, "\n"); + Output(Out, "BOOLEAN\n"); + Output(Out, "DIB_%uBPP_BitBlt(PBLTINFO BltInfo)\n", Bpp); + Output(Out, "{\n"); + Output(Out, "PrimitivesTable[BltInfo->Rop4 & 0xff](BltInfo);\n"); + Output(Out, "\n"); + Output(Out, "return TRUE;\n"); + Output(Out, "}\n"); } static void Generate(char *OutputDir, unsigned Bpp) { - FILE *Out; - unsigned RopCode; - PROPINFO RopInfo; - char *FileName; - - FileName = malloc(strlen(OutputDir) + 12); - if (NULL == FileName) - { - fprintf(stderr, "Out of memory\n"); - exit(1); - } - strcpy(FileName, OutputDir); - if ('/' != FileName[strlen(FileName) - 1]) - { - strcat(FileName, "/"); - } - sprintf(FileName + strlen(FileName), "dib%ugen.c", Bpp); - - Out = fopen(FileName, "w"); - free(FileName); - if (NULL == Out) - { - perror("Error opening output file"); - exit(1); - } - - MARK(Out); - Output(Out, "/* This is a generated file. Please do not edit */\n"); - Output(Out, "\n"); - Output(Out, "#include <win32k.h>\n"); - CreateShiftTables(Out); - - RopInfo = FindRopInfo(ROPCODE_GENERIC); - CreatePrimitive(Out, Bpp, RopInfo); - for (RopCode = 0; RopCode < 256; RopCode++) - { - RopInfo = FindRopInfo(RopCode); - if (NULL != RopInfo) - { - CreatePrimitive(Out, Bpp, RopInfo); - } - } - CreateTable(Out, Bpp); - CreateBitBlt(Out, Bpp); - - fclose(Out); + FILE *Out; + unsigned RopCode; + PROPINFO RopInfo; + char *FileName; + + FileName = malloc(strlen(OutputDir) + 12); + if (NULL == FileName) + { + fprintf(stderr, "Out of memory\n"); + exit(1); + } + strcpy(FileName, OutputDir); + if ('/' != FileName[strlen(FileName) - 1]) + { + strcat(FileName, "/"); + } + sprintf(FileName + strlen(FileName), "dib%ugen.c", Bpp); + + Out = fopen(FileName, "w"); + free(FileName); + if (NULL == Out) + { + perror("Error opening output file"); + exit(1); + } + + MARK(Out); + Output(Out, "/* This is a generated file. Please do not edit */\n"); + Output(Out, "\n"); + Output(Out, "#include <win32k.h>\n"); + CreateShiftTables(Out); + + RopInfo = FindRopInfo(ROPCODE_GENERIC); + CreatePrimitive(Out, Bpp, RopInfo); + for (RopCode = 0; RopCode < 256; RopCode++) + { + RopInfo = FindRopInfo(RopCode); + if (NULL != RopInfo) + { + CreatePrimitive(Out, Bpp, RopInfo); + } + } + CreateTable(Out, Bpp); + CreateBitBlt(Out, Bpp); + + fclose(Out); } int main(int argc, char *argv[]) { - unsigned Index; - static unsigned DestBpp[] = + unsigned Index; + static unsigned DestBpp[] = { 8, 16, 32 }; - if (argc < 2) + if (argc < 2) + return 0; + + for (Index = 0; Index < sizeof(DestBpp) / sizeof(DestBpp[0]); Index++) + { + Generate(argv[1], DestBpp[Index]); + } + return 0; - - for (Index = 0; Index < sizeof(DestBpp) / sizeof(DestBpp[0]); Index++) - { - Generate(argv[1], DestBpp[Index]); - } - - return 0; -} +}

12 years, 8 months

[pschweitzer] 55996: [NTOSKRNL] Comment out two overzealous ASSERTs

by pschweitzer＠svn.reactos.org

Author: pschweitzer Date: Sun Mar 4 12:52:02 2012 New Revision: 55996 URL: http://svn.reactos.org/svn/reactos?rev=55996&view=rev Log: [NTOSKRNL] Comment out two overzealous ASSERTs Modified: trunk/reactos/ntoskrnl/fsrtl/filelock.c Modified: trunk/reactos/ntoskrnl/fsrtl/filelock.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/fsrtl/filelock.c?… ============================================================================== --- trunk/reactos/ntoskrnl/fsrtl/filelock.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/fsrtl/filelock.c [iso-8859-1] Sun Mar 4 12:52:02 2012 @@ -251,7 +251,9 @@ BOOLEAN InsertedNew; DPRINT1("FsRtlPrivateLock() is stubplemented!\n"); - ASSERT(AlreadySynchronized); + /* Windows 2003 ignores that parameter + ASSERT(AlreadySynchronized); + */ /* Initialize the lock, if necessary */ if (!FileLock->LockInformation) @@ -502,7 +504,8 @@ PLOCK_INFORMATION InternalInfo = FileLock->LockInformation; // The region to unlock must correspond exactly to a previously locked region // -- msdn - ASSERT(AlreadySynchronized); + // But Windows 2003 doesn't assert on it and simply ignores that parameter + // ASSERT(AlreadySynchronized); Find.Exclusive.FileLock.StartingByte = *FileOffset; Find.Exclusive.FileLock.EndingByte.QuadPart = FileOffset->QuadPart + Length->QuadPart;

12 years, 8 months

[tkreuzer] 55995: [WIN32K] Fix build 2

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sun Mar 4 12:01:59 2012 New Revision: 55995 URL: http://svn.reactos.org/svn/reactos?rev=55995&view=rev Log: [WIN32K] Fix build 2 Modified: trunk/reactos/subsystems/win32/win32k/objects/line.c Modified: trunk/reactos/subsystems/win32/win32k/objects/line.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/line.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/line.c [iso-8859-1] Sun Mar 4 12:01:59 2012 @@ -419,7 +419,8 @@ PDC dc; PDC_ATTR pdcattr; POINT *line_pts = NULL, *line_pts_old, *bzr_pts = NULL, bzr[4]; - ULONG i, num_pts, num_bzr_pts, space, space_old, size; + INT num_pts, num_bzr_pts, space, space_old, size; + ULONG i; BOOL result = FALSE; dc = DC_LockDc(hdc);

12 years, 8 months

[tkreuzer] 55994: [WIN32K] Fix build

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sun Mar 4 11:43:08 2012 New Revision: 55994 URL: http://svn.reactos.org/svn/reactos?rev=55994&view=rev Log: [WIN32K] Fix build Modified: trunk/reactos/subsystems/win32/win32k/ntuser/class.c Modified: trunk/reactos/subsystems/win32/win32k/ntuser/class.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/class.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/class.c [iso-8859-1] Sun Mar 4 11:43:08 2012 @@ -1634,8 +1634,8 @@ TRACE("SetClassLong(%d, %x)\n", Index, NewLong); - if ((Index + (INT)sizeof(ULONG_PTR)) < Index || - (Index + (INT)sizeof(ULONG_PTR)) > Class->cbclsExtra) + if (((ULONG)Index + sizeof(ULONG_PTR)) < (ULONG)Index || + ((ULONG)Index + sizeof(ULONG_PTR)) > (ULONG)Class->cbclsExtra) { EngSetLastError(ERROR_INVALID_PARAMETER); return 0;

12 years, 8 months

[tkreuzer] 55993: [WIN32K] - Fix a possible integer overflow in NtGdiCreateBitmap - Don't cast length of LARGE_STRING (ULONG) to USHORT, instead truncate to MAXUSHORT. This is still hacky, but bett...

by tkreuzer＠svn.reactos.org

Author: tkreuzer Date: Sun Mar 4 11:32:43 2012 New Revision: 55993 URL: http://svn.reactos.org/svn/reactos?rev=55993&view=rev Log: [WIN32K] - Fix a possible integer overflow in NtGdiCreateBitmap - Don't cast length of LARGE_STRING (ULONG) to USHORT, instead truncate to MAXUSHORT. This is still hacky, but better than before. - Fix MSVC warnings Modified: trunk/reactos/include/reactos/win32k/ntuser.h trunk/reactos/subsystems/win32/win32k/include/userfuncs.h trunk/reactos/subsystems/win32/win32k/ntuser/class.c trunk/reactos/subsystems/win32/win32k/ntuser/defwnd.c trunk/reactos/subsystems/win32/win32k/ntuser/display.c trunk/reactos/subsystems/win32/win32k/ntuser/input.c trunk/reactos/subsystems/win32/win32k/ntuser/kbdlayout.c trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c trunk/reactos/subsystems/win32/win32k/ntuser/menu.c trunk/reactos/subsystems/win32/win32k/ntuser/message.c trunk/reactos/subsystems/win32/win32k/ntuser/metric.c trunk/reactos/subsystems/win32/win32k/ntuser/msgqueue.c trunk/reactos/subsystems/win32/win32k/ntuser/painting.c trunk/reactos/subsystems/win32/win32k/ntuser/scrollbar.c trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c trunk/reactos/subsystems/win32/win32k/ntuser/window.c trunk/reactos/subsystems/win32/win32k/ntuser/winpos.c trunk/reactos/subsystems/win32/win32k/objects/arc.c trunk/reactos/subsystems/win32/win32k/objects/bitmaps.c trunk/reactos/subsystems/win32/win32k/objects/dibobj.c trunk/reactos/subsystems/win32/win32k/objects/drawing.c trunk/reactos/subsystems/win32/win32k/objects/fillshap.c trunk/reactos/subsystems/win32/win32k/objects/gdiobj.c trunk/reactos/subsystems/win32/win32k/objects/line.c trunk/reactos/subsystems/win32/win32k/objects/palette.c trunk/reactos/subsystems/win32/win32k/objects/path.c trunk/reactos/subsystems/win32/win32k/objects/pen.c Modified: trunk/reactos/include/reactos/win32k/ntuser.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/win32k/ntu… ============================================================================== --- trunk/reactos/include/reactos/win32k/ntuser.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/win32k/ntuser.h [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -2038,7 +2038,7 @@ UINT NTAPI NtUserGetKeyboardLayoutList( - INT nItems, + ULONG nItems, HKL *pHklBuff); BOOL Modified: trunk/reactos/subsystems/win32/win32k/include/userfuncs.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/in… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/include/userfuncs.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/include/userfuncs.h [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -62,8 +62,8 @@ /*************** METRIC.C ***************/ -BOOL FASTCALL InitMetrics(VOID); -ULONG FASTCALL UserGetSystemMetrics(ULONG Index); +BOOL NTAPI InitMetrics(VOID); +LONG NTAPI UserGetSystemMetrics(ULONG Index); /*************** KEYBOARD.C ***************/ Modified: trunk/reactos/subsystems/win32/win32k/ntuser/class.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/class.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/class.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -64,7 +64,7 @@ NULL, FNID_MESSAGEWND, ICLS_HWNDMESSAGE - } + } }; static struct @@ -82,8 +82,8 @@ { FNID_BUTTON, ICLS_BUTTON}, { FNID_COMBOBOX, ICLS_COMBOBOX}, { FNID_COMBOLBOX, ICLS_COMBOLBOX}, - { FNID_DIALOG, ICLS_DIALOG}, - { FNID_EDIT, ICLS_EDIT}, + { FNID_DIALOG, ICLS_DIALOG}, + { FNID_EDIT, ICLS_EDIT}, { FNID_LISTBOX, ICLS_LISTBOX}, { FNID_MDICLIENT, ICLS_MDICLIENT}, { FNID_STATIC, ICLS_STATIC}, @@ -97,7 +97,7 @@ LookupFnIdToiCls(int FnId, int *iCls ) { int i; - + for ( i = 0; i < ARRAYSIZE(FnidToiCls); i++) { if (FnidToiCls[i].FnId == FnId) @@ -178,7 +178,7 @@ { PCLS Class; PPROCESSINFO pi = (PPROCESSINFO)Process; - + if (pi != NULL) { /* Free all local classes */ @@ -413,7 +413,7 @@ Class->lpfnWndProc = chWndProc; Class->Unicode = TRUE; Class->CSF_flags &= ~CSF_ANSIPROC; - Class->CSF_flags |= CSF_SERVERSIDEPROC; + Class->CSF_flags |= CSF_SERVERSIDEPROC; } else { @@ -433,7 +433,7 @@ { Class->Unicode = !Ansi; Class->lpfnWndProc = chWndProc; - + Class = Class->pclsNext; } @@ -936,7 +936,7 @@ ANSI_STRING AnsiString; Class->lpszClientAnsiMenuName = (PSTR)pszMenuNameBuffer; - AnsiString.MaximumLength = RtlUnicodeStringToAnsiSize(MenuName); + AnsiString.MaximumLength = (USHORT)RtlUnicodeStringToAnsiSize(MenuName); AnsiString.Buffer = Class->lpszClientAnsiMenuName; Status = RtlUnicodeStringToAnsiString(&AnsiString, MenuName, @@ -1525,7 +1525,7 @@ ANSI_STRING AnsiString; PWSTR strBufW; - AnsiString.MaximumLength = RtlUnicodeStringToAnsiSize(MenuName); + AnsiString.MaximumLength = (USHORT)RtlUnicodeStringToAnsiSize(MenuName); strBufW = UserHeapAlloc(MenuName->Length + sizeof(UNICODE_NULL) + AnsiString.MaximumLength); @@ -1634,8 +1634,8 @@ TRACE("SetClassLong(%d, %x)\n", Index, NewLong); - if (Index + sizeof(ULONG_PTR) < Index || - Index + sizeof(ULONG_PTR) > Class->cbclsExtra) + if ((Index + (INT)sizeof(ULONG_PTR)) < Index || + (Index + (INT)sizeof(ULONG_PTR)) > Class->cbclsExtra) { EngSetLastError(ERROR_INVALID_PARAMETER); return 0; @@ -1822,7 +1822,7 @@ lpwcx->style &= ~CS_GLOBALCLASS; lpwcx->lpfnWndProc = IntGetClassWndProc(Class, Ansi); - + lpwcx->cbClsExtra = Class->cbclsExtra; lpwcx->cbWndExtra = Class->cbwndExtra; lpwcx->hIcon = Class->hIcon; /* FIXME: Get handle from pointer */ @@ -1840,7 +1840,7 @@ * lpszClientXxxMenuName should already be mapped to user space. */ /* Copy string ptr to user. */ - if ( Class->lpszClientUnicodeMenuName != NULL && + if ( Class->lpszClientUnicodeMenuName != NULL && Class->MenuNameIsString) { lpwcx->lpszMenuName = UserHeapAddressToUser(Ansi ? @@ -1897,7 +1897,7 @@ ClassName.Length = 0; ClassName.MaximumLength = 0; } - + wc.cbSize = sizeof(wc); wc.style = DefaultServerClasses[i].Style; @@ -2016,15 +2016,14 @@ CapturedMenuName = ProbeForReadUnicodeString(pClassMenuName->pusMenuName); - if ( CapturedName.Length & 1 || - CapturedMenuName.Length & 1 || - CapturedClassInfo.cbClsExtra < 0 || - CapturedClassInfo.cbClsExtra + - CapturedName.Length + - CapturedMenuName.Length + - sizeof(CLS) < CapturedClassInfo.cbClsExtra || - CapturedClassInfo.cbWndExtra < 0 || - CapturedClassInfo.hInstance == NULL) + if ( (CapturedName.Length & 1) || + (CapturedMenuName.Length & 1) || + (CapturedClassInfo.cbClsExtra < 0) || + ((CapturedClassInfo.cbClsExtra + CapturedName.Length + + CapturedMenuName.Length + sizeof(CLS)) + < (ULONG)CapturedClassInfo.cbClsExtra) || + (CapturedClassInfo.cbWndExtra < 0) || + (CapturedClassInfo.hInstance == NULL) ) { ERR("NtUserRegisterClassExWOW Invalid Parameter Error!\n"); goto InvalidParameter; Modified: trunk/reactos/subsystems/win32/win32k/ntuser/defwnd.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/defwnd.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/defwnd.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -59,7 +59,7 @@ break; } } - else + else { co_IntSendMessage(WndChild->head.h, WM_ENDSESSION, KillTimers, lParams); if (KillTimers) @@ -82,7 +82,7 @@ lResult = MCSR_DONOTSHUTDOWN; } } - else + else { co_IntSendMessage(pWindow->head.h, WM_ENDSESSION, KillTimers, lParams); if (KillTimers) @@ -141,7 +141,7 @@ { Hook = TRUE; lResult = co_HOOK_CallHooks(WH_CBT, HCBT_SYSCOMMAND, wParam, lParam); - + if (lResult) return lResult; } @@ -218,7 +218,7 @@ return (LRESULT) DefWndControlColor((HDC)wParam, HIWORD(lParam)); case WM_GETHOTKEY: - return DefWndGetHotKey(UserHMGetHandle(Wnd)); + return DefWndGetHotKey(UserHMGetHandle(Wnd)); case WM_SETHOTKEY: return DefWndSetHotKey(Wnd, wParam); Modified: trunk/reactos/subsystems/win32/win32k/ntuser/display.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/display.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/display.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -102,8 +102,8 @@ /* Initialize the UNICODE_STRING */ ustrDisplayDrivers.Buffer = awcBuffer; - ustrDisplayDrivers.MaximumLength = cbSize; - ustrDisplayDrivers.Length = cbSize; + ustrDisplayDrivers.MaximumLength = (USHORT)cbSize; + ustrDisplayDrivers.Length = (USHORT)cbSize; /* Set Buffer for description and size of remaining buffer */ ustrDescription.Buffer = awcBuffer + (cbSize / sizeof(WCHAR)); @@ -117,8 +117,8 @@ &cbSize); if (NT_SUCCESS(Status)) { - ustrDescription.MaximumLength = cbSize; - ustrDescription.Length = cbSize; + ustrDescription.MaximumLength = (USHORT)cbSize; + ustrDescription.Length = (USHORT)cbSize; } else { Modified: trunk/reactos/subsystems/win32/win32k/ntuser/input.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/input.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/input.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -137,7 +137,7 @@ ULONG cWaitObjects = 0, cMaxWaitObjects = 1; MOUSE_INPUT_DATA MouseInput; KEYBOARD_INPUT_DATA KeyInput; - + ByteOffset.QuadPart = (LONGLONG)0; //WaitTimeout.QuadPart = (LONGLONG)(-10000000); @@ -196,7 +196,7 @@ &ByteOffset, NULL); } - + if (MouStatus == STATUS_PENDING) WaitObjects[cWaitObjects++] = &pMouDevice->Event; } @@ -215,7 +215,7 @@ sizeof(KEYBOARD_INPUT_DATA), &ByteOffset, NULL); - + } if (KbdStatus == STATUS_PENDING) WaitObjects[cWaitObjects++] = &pKbdDevice->Event; @@ -233,7 +233,8 @@ NULL,//&WaitTimeout, NULL); - if (Status >= STATUS_WAIT_0 && Status < STATUS_WAIT_0 + cWaitObjects) + if ((Status >= STATUS_WAIT_0) && + (Status < (STATUS_WAIT_0 + (LONG)cWaitObjects))) { /* Some device has finished reading */ pSignaledObject = WaitObjects[Status - STATUS_WAIT_0]; Modified: trunk/reactos/subsystems/win32/win32k/ntuser/kbdlayout.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/kbdlayout.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/kbdlayout.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -440,7 +440,7 @@ UINT APIENTRY NtUserGetKeyboardLayoutList( - INT nBuff, + ULONG nBuff, HKL *pHklBuff) { UINT uRet = 0; @@ -600,7 +600,7 @@ pklLast = gspklBaseLayout->pklPrev; while (pklLast != gspklBaseLayout && pklLast->dwKL_Flags & KLF_UNLOAD) pklLast = pklLast->pklPrev; - + /* Add new layout to the list */ pKl->pklNext = pklLast->pklNext; pKl->pklPrev = pklLast; Modified: trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/keyboard.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -876,8 +876,8 @@ Msg.lParam |= KF_REPEAT << 16; if (!bIsDown) Msg.lParam |= KF_UP << 16; - /* FIXME: Set KF_DLGMODE and KF_MENUMODE when needed */ - if (pFocusQueue->QF_flags & QF_DIALOGACTIVE) + /* FIXME: Set KF_DLGMODE and KF_MENUMODE when needed */ + if (pFocusQueue->QF_flags & QF_DIALOGACTIVE) Msg.lParam |= KF_DLGMODE << 16; if (pFocusQueue->MenuOwner) // pFocusQueue->MenuState) // MenuState needs a start flag... Msg.lParam |= KF_MENUMODE << 16; @@ -967,7 +967,7 @@ return ProcessKeyEvent(wVk, wScanCode, pKbdInput->dwFlags, bInjected, dwTime, pKbdInput->dwExtraInfo); } -/* +/* * UserProcessKeyboardInput * * Process raw keyboard input data @@ -1045,7 +1045,7 @@ } } -/* +/* * IntTranslateKbdMessage * * Addes WM_(SYS)CHAR messages to message queue if message @@ -1408,8 +1408,8 @@ if (pKeyName) { cchKeyName = wcslen(pKeyName); - if (cchKeyName > cchSize - 1) - cchKeyName = cchSize - 1; // Don't count '\0' + if (cchKeyName > (cchSize - 1UL)) + cchKeyName = cchSize - 1UL; // Don't count '\0' _SEH2_TRY { Modified: trunk/reactos/subsystems/win32/win32k/ntuser/menu.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/menu.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/menu.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -804,7 +804,7 @@ IntInsertMenuItem(PMENU_OBJECT MenuObject, UINT uItem, BOOL fByPosition, PROSMENUITEMINFO ItemInfo) { - int pos = (int)uItem; + int pos; PMENU_ITEM MenuItem; PMENU_OBJECT SubMenu = NULL; @@ -818,7 +818,8 @@ { SubMenu = MenuObject; /* calculate position */ - if(MenuObject->MenuInfo.MenuItemCount < pos) + pos = (int)uItem; + if(uItem > MenuObject->MenuInfo.MenuItemCount) { pos = MenuObject->MenuInfo.MenuItemCount; } @@ -1254,7 +1255,7 @@ bti->rcTitleBar.right = pWindowObject->rcWindow.right - pWindowObject->rcWindow.left; bti->rcTitleBar.bottom = pWindowObject->rcWindow.bottom - pWindowObject->rcWindow.top; - /* Is it iconiced ? */ + /* Is it iconiced ? */ if ((dwStyle & WS_ICONIC)!=WS_ICONIC) { /* Remove frame from rectangle */ @@ -1303,24 +1304,24 @@ /* FIXME: Note this value should exists in pWindowObject for UserGetSystemMetrics(SM_CYSMCAPTION) */ bti->rcTitleBar.bottom += UserGetSystemMetrics(SM_CYSMCAPTION); } - else + else { /* FIXME: Note this value should exists in pWindowObject for UserGetSystemMetrics(SM_CYCAPTION) and UserGetSystemMetrics(SM_CXSIZE) */ bti->rcTitleBar.bottom += UserGetSystemMetrics(SM_CYCAPTION); bti->rcTitleBar.left += UserGetSystemMetrics(SM_CXSIZE); } - if (dwStyle & WS_CAPTION) + if (dwStyle & WS_CAPTION) { bti->rgstate[1] = STATE_SYSTEM_INVISIBLE; - if (dwStyle & WS_SYSMENU) + if (dwStyle & WS_SYSMENU) { - if (!(dwStyle & (WS_MINIMIZEBOX|WS_MAXIMIZEBOX))) + if (!(dwStyle & (WS_MINIMIZEBOX|WS_MAXIMIZEBOX))) { bti->rgstate[2] = STATE_SYSTEM_INVISIBLE; bti->rgstate[3] = STATE_SYSTEM_INVISIBLE; } - else + else { if (!(dwStyle & WS_MINIMIZEBOX)) { @@ -1341,7 +1342,7 @@ bti->rgstate[5] = STATE_SYSTEM_UNAVAILABLE; } } - else + else { bti->rgstate[2] = STATE_SYSTEM_INVISIBLE; bti->rgstate[3] = STATE_SYSTEM_INVISIBLE; @@ -1609,13 +1610,13 @@ } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { - /* Fail copy the data */ + /* Fail copy the data */ EngSetLastError(ERROR_INVALID_PARAMETER); retValue = FALSE; } _SEH2_END - /* Get the tile bar info */ + /* Get the tile bar info */ if (retValue) { retValue = intGetTitleBarInfo(WindowObject, &bartitleinfo); @@ -1629,7 +1630,7 @@ } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { - /* Fail copy the data */ + /* Fail copy the data */ EngSetLastError(ERROR_INVALID_PARAMETER); retValue = FALSE; } @@ -2240,7 +2241,7 @@ { EngSetLastError(ERROR_INVALID_PARAMETER); // This will crash menu (line 80) correct_behavior test! -// "NT4 and below can't handle a bigger MENUITEMINFO struct" +// "NT4 and below can't handle a bigger MENUITEMINFO struct" //EngSetLastError(ERROR_MENU_ITEM_NOT_FOUND); return( FALSE); } @@ -2445,7 +2446,7 @@ ERR("Failed to capture MenuItem Caption (status 0x%08x)\n",Status); SetLastNtError(Status); RETURN(FALSE); - } + } } if (bInsert) RETURN( UserInsertMenuItem(Menu, uItem, fByPosition, lpmii)); Modified: trunk/reactos/subsystems/win32/win32k/ntuser/message.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/message.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/message.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -1844,7 +1844,7 @@ { MSG msg; RECT rect; - WORD wDragWidth, wDragHeight; + ULONG wDragWidth, wDragHeight; DECLARE_RETURN(BOOL); TRACE("Enter NtUserDragDetect(%x)\n", hWnd); Modified: trunk/reactos/subsystems/win32/win32k/ntuser/metric.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/metric.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/metric.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -15,7 +15,7 @@ /* FUNCTIONS *****************************************************************/ BOOL -FASTCALL +NTAPI InitMetrics(VOID) { INT *piSysMet = gpsi->aiSysMet; @@ -175,7 +175,8 @@ return TRUE; } -ULONG FASTCALL +LONG +NTAPI UserGetSystemMetrics(ULONG Index) { ASSERT(gpsi); Modified: trunk/reactos/subsystems/win32/win32k/ntuser/msgqueue.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/msgqueue.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/msgqueue.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -94,7 +94,7 @@ if ((pWnd->style & WS_VISIBLE) && IntPtInWindow(pWnd, x, y)) return pWnd; } - + /* Window has not been found */ return NULL; } @@ -177,10 +177,10 @@ pti = PsGetCurrentThreadWin32Thread(); MessageQueue = pti->MessageQueue; - + /* Update counter */ MessageQueue->ShowingCursor += bShow ? 1 : -1; - + /* Check for trivial cases */ if ((bShow && MessageQueue->ShowingCursor != 0) || (!bShow && MessageQueue->ShowingCursor != -1)) @@ -189,7 +189,7 @@ internally to check if cursor is visible */ return MessageQueue->ShowingCursor; } - + /* Check if cursor is above window owned by this MessageQueue */ pWnd = IntTopLevelWindowFromPoint(gpsi->ptCursor.x, gpsi->ptCursor.y); if (pWnd && pWnd->head.pti->MessageQueue == MessageQueue) @@ -206,7 +206,7 @@ GreMovePointer(hdcScreen, -1, -1); TRACE("Removing pointer!\n"); } - + /* Update global info */ IntGetSysCursorInfo()->ShowingCursor = MessageQueue->ShowingCursor; } @@ -1382,7 +1382,7 @@ if ((msg->message == clk_msg.message) && (msg->hwnd == clk_msg.hwnd) && (msg->wParam == clk_msg.wParam) && - (msg->time - clk_msg.time < gspv.iDblClickTime) && + ((msg->time - clk_msg.time) < (ULONG)gspv.iDblClickTime) && (abs(msg->pt.x - clk_msg.pt.x) < UserGetSystemMetrics(SM_CXDOUBLECLK)/2) && (abs(msg->pt.y - clk_msg.pt.y) < UserGetSystemMetrics(SM_CYDOUBLECLK)/2)) { @@ -1999,7 +1999,7 @@ MessageQueue->nCntsQBits[QSRosPostMessage] = 0; MessageQueue->nCntsQBits[QSRosSendMessage] = 0; MessageQueue->nCntsQBits[QSRosHotKey] = 0; - + if (MessageQueue->CursorObject) { PCURICON_OBJECT pCursor = MessageQueue->CursorObject; @@ -2020,7 +2020,7 @@ UserDereferenceObject(pCursor); } - + } PUSER_MESSAGE_QUEUE FASTCALL @@ -2183,7 +2183,7 @@ UserLeave(); - return Ret; + return (SHORT)Ret; } Modified: trunk/reactos/subsystems/win32/win32k/ntuser/painting.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/painting.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/painting.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -777,7 +777,7 @@ } BOOL -FASTCALL +FASTCALL IntFlashWindowEx(PWND pWnd, PFLASHWINFO pfwi) { PPROPERTY pprop; @@ -1831,9 +1831,9 @@ else if (pWnd != NULL) // FIXME: Windows does not do that { UNICODE_STRING ustr; - ustr.Buffer = pWnd->strName.Buffer; - ustr.Length = pWnd->strName.Length; - ustr.MaximumLength = pWnd->strName.MaximumLength; + ustr.Buffer = pWnd->strName.Buffer; // FIXME: LARGE_STRING truncated! + ustr.Length = (USHORT)min(pWnd->strName.Length, MAXUSHORT); + ustr.MaximumLength = (USHORT)min(pWnd->strName.MaximumLength, MAXUSHORT); UserDrawCaptionText(hDc, &ustr, &Rect, uFlags, hFont); } } Modified: trunk/reactos/subsystems/win32/win32k/ntuser/scrollbar.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/scrollbar.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/scrollbar.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -361,7 +361,7 @@ /* Make sure the page size is valid */ if (Info->nPage < 0) Info->nPage = 0; - else if (Info->nMax - Info->nMin + 1 < Info->nPage) + else if ((Info->nMax - Info->nMin + 1UL) < Info->nPage) { Info->nPage = Info->nMax - Info->nMin + 1; } @@ -371,7 +371,7 @@ { Info->nPos = Info->nMin; } - else if (Info->nPos > Info->nMax - max(Info->nPage - 1, 0)) + else if (Info->nPos > (Info->nMax - max((int)Info->nPage - 1, 0))) { Info->nPos = Info->nMax - max(Info->nPage - 1, 0); } @@ -421,7 +421,7 @@ done: if ( action & SA_SSI_HIDE ) - { + { co_UserShowScrollBar(Window, nBar, FALSE, FALSE); } else @@ -616,7 +616,7 @@ case SB_HORZ: if (fShowH) set_bits |= WS_HSCROLL; else clear_bits |= WS_HSCROLL; - if( nBar == SB_HORZ ) break; + if( nBar == SB_HORZ ) break; /* Fall through */ case SB_VERT: if (fShowV) set_bits |= WS_VSCROLL; @@ -701,8 +701,8 @@ BOOL APIENTRY NtUserSBGetParms( - HWND hWnd, - int fnBar, + HWND hWnd, + int fnBar, PSBDATA pSBData, LPSCROLLINFO lpsi) { @@ -822,11 +822,11 @@ if(InfoH) Chg = (IntEnableScrollBar(TRUE, InfoH, wArrows) || Chg); - + ERR("FIXME: EnableScrollBar wSBflags %d wArrows %d Chg %d\n",wSBflags,wArrows, Chg); // Done in user32: // SCROLL_RefreshScrollBar( hwnd, nBar, TRUE, TRUE ); - + if (OrigArrows == wArrows) RETURN( FALSE); RETURN( TRUE); Modified: trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/sysparams.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -727,7 +727,7 @@ UserRefObjectCo(pwndCurrent, &Ref); co_WinPosSetWindowPos(pwndCurrent, 0, pwndCurrent->rcWindow.left,pwndCurrent->rcWindow.top, pwndCurrent->rcWindow.right-pwndCurrent->rcWindow.left - ,pwndCurrent->rcWindow.bottom - pwndCurrent->rcWindow.top, + ,pwndCurrent->rcWindow.bottom - pwndCurrent->rcWindow.top, SWP_FRAMECHANGED|SWP_NOACTIVATE|SWP_NOCOPYBITS| SWP_NOMOVE|SWP_NOZORDER|SWP_NOREDRAW); UserDerefObjectCo(pwndCurrent); @@ -800,7 +800,7 @@ return SpiSetInt(&gspv.uiGridGranularity, uiParam, KEY_DESKTOP, VAL_GRID, fl); case SPI_GETDESKWALLPAPER: - uiParam = min(uiParam, gspv.ustrWallpaper.Length + 1); + uiParam = min(uiParam, gspv.ustrWallpaper.Length + 1UL); return SpiGet(pvParam, gspv.awcWallpaper, uiParam, fl); case SPI_SETDESKWALLPAPER: @@ -1590,7 +1590,7 @@ } ulResult = 1; } - + return ulResult; } @@ -1616,7 +1616,7 @@ TRACE("Leave NtUserSystemParametersInfo, returning %d\n", bResult); UserLeave(); - + return bResult; } Modified: trunk/reactos/subsystems/win32/win32k/ntuser/window.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/window.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/window.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -795,7 +795,7 @@ Rect->bottom = UserGetSystemMetrics(SM_CYMINIMIZED); return; } - if ( Wnd != UserGetDesktopWindow()) // Wnd->fnid != FNID_DESKTOP ) + if ( Wnd != UserGetDesktopWindow()) // Wnd->fnid != FNID_DESKTOP ) { *Rect = Wnd->rcClient; RECTL_vOffsetRect(Rect, -Wnd->rcClient.left, -Wnd->rcClient.top); @@ -1953,7 +1953,7 @@ if (Window->state & WNDS_ANSICREATOR) { ANSI_STRING AnsiString; - AnsiString.MaximumLength = RtlUnicodeStringToAnsiSize(ClassName)+sizeof(CHAR); + AnsiString.MaximumLength = (USHORT)RtlUnicodeStringToAnsiSize(ClassName)+sizeof(CHAR); pszClass = UserHeapAlloc(AnsiString.MaximumLength); if (!pszClass) { @@ -1984,13 +1984,13 @@ { UNICODE_STRING Name; Name.Buffer = WindowName->Buffer; - Name.Length = WindowName->Length; - Name.MaximumLength = WindowName->MaximumLength; + Name.Length = (USHORT)min(WindowName->Length, MAXUSHORT); // FIXME: LARGE_STRING truncated + Name.MaximumLength = (USHORT)min(WindowName->MaximumLength, MAXUSHORT); if (Window->state & WNDS_ANSICREATOR) { ANSI_STRING AnsiString; - AnsiString.MaximumLength = RtlUnicodeStringToAnsiSize(&Name) + sizeof(CHAR); + AnsiString.MaximumLength = (USHORT)RtlUnicodeStringToAnsiSize(&Name) + sizeof(CHAR); pszName = UserHeapAlloc(AnsiString.MaximumLength); if (!pszName) { @@ -2325,8 +2325,8 @@ /* We pass it on as a UNICODE_STRING */ ustrClassName.Buffer = lstrClassName.Buffer; - ustrClassName.Length = lstrClassName.Length; - ustrClassName.MaximumLength = lstrClassName.MaximumLength; + ustrClassName.Length = (USHORT)min(lstrClassName.Length, MAXUSHORT); // FIXME: LARGE_STRING truncated + ustrClassName.MaximumLength = (USHORT)min(lstrClassName.MaximumLength, MAXUSHORT); } /* Fill the CREATESTRUCTW */ @@ -2593,10 +2593,10 @@ send WM_GETTEXT messages to windows belonging to its processes */ if (!ClassAtom || Child->pcls->atomClassName == ClassAtom) { - // HACK: use UNICODE_STRING instead of LARGE_STRING + // FIXME: LARGE_STRING truncated CurrentWindowName.Buffer = Child->strName.Buffer; - CurrentWindowName.Length = Child->strName.Length; - CurrentWindowName.MaximumLength = Child->strName.MaximumLength; + CurrentWindowName.Length = (USHORT)min(Child->strName.Length, MAXUSHORT); + CurrentWindowName.MaximumLength = (USHORT)min(Child->strName.MaximumLength, MAXUSHORT); if(!CheckWindowName || (Child->strName.Length < 0xFFFF && !RtlCompareUnicodeString(WindowName, &CurrentWindowName, TRUE))) @@ -2772,8 +2772,8 @@ The user mode version however calls GetWindowText() which will send WM_GETTEXT messages to windows belonging to its processes */ ustr.Buffer = TopLevelWindow->strName.Buffer; - ustr.Length = TopLevelWindow->strName.Length; - ustr.MaximumLength = TopLevelWindow->strName.MaximumLength; + ustr.Length = (USHORT)min(TopLevelWindow->strName.Length, MAXUSHORT); // FIXME:LARGE_STRING truncated + ustr.MaximumLength = (USHORT)min(TopLevelWindow->strName.MaximumLength, MAXUSHORT); WindowMatches = !CheckWindowName || (TopLevelWindow->strName.Length < 0xFFFF && !RtlCompareUnicodeString(&WindowName, &ustr, TRUE)); @@ -3158,7 +3158,7 @@ WinStaObject->ShellListView = hwndListView; ti = GetW32ThreadInfo(); - if (ti->pDeskInfo) + if (ti->pDeskInfo) { ti->pDeskInfo->hShellWindow = hwndShell; ti->pDeskInfo->ppiShellProcess = ti->ppi; @@ -3450,7 +3450,7 @@ case GWL_ID: case GWL_HINSTANCE: case GWL_HWNDPARENT: - RETURN( co_UserSetWindowLong(Window->head.h, Index, (UINT)NewValue, TRUE)); + RETURN( (WORD)co_UserSetWindowLong(Window->head.h, Index, (UINT)NewValue, TRUE)); default: if (Index < 0) { @@ -3459,7 +3459,7 @@ } } - if (Index > Window->cbwndExtra - sizeof(WORD)) + if ((ULONG)Index > (Window->cbwndExtra - sizeof(WORD))) { EngSetLastError(ERROR_INVALID_PARAMETER); RETURN( 0); Modified: trunk/reactos/subsystems/win32/win32k/ntuser/winpos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/nt… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/ntuser/winpos.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/ntuser/winpos.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -173,7 +173,7 @@ TRACE("X:%d Y:%d XS:%d YS:%d\n",x,y,xspacing,yspacing); - for( i = 0; List[i]; i++) + for(i = 0; List[i]; i++) { PWND Child; @@ -197,7 +197,7 @@ UserDerefObjectCo(Child); - if (x <= rectParent.right - UserGetSystemMetrics(SM_CXMINSPACING)) + if (x <= (rectParent.right - UserGetSystemMetrics(SM_CXMINSPACING))) x += xspacing; else { @@ -228,8 +228,8 @@ } IntGetClientRect( pwndParent, &rectParent ); - if ((Pos->x >= rectParent.left) && (Pos->x + UserGetSystemMetrics(SM_CXICON) < rectParent.right) && - (Pos->y >= rectParent.top) && (Pos->y + UserGetSystemMetrics(SM_CYICON) < rectParent.bottom)) + if ((Pos->x >= rectParent.left) && ((Pos->x + UserGetSystemMetrics(SM_CXICON)) < rectParent.right) && + (Pos->y >= rectParent.top) && ((Pos->y + UserGetSystemMetrics(SM_CYICON)) < rectParent.bottom)) return; /* The icon already has a suitable position */ xspacing = UserGetSystemMetrics(SM_CXICONSPACING); @@ -318,7 +318,7 @@ if (Wnd->state & WNDS_MAXIMIZESTOMONITOR) { Wnd->InternalPos.flags &= ~WPF_MAXINIT; - Wnd->InternalPos.MaxPos.x = Wnd->InternalPos.MaxPos.y = -1; + Wnd->InternalPos.MaxPos.x = Wnd->InternalPos.MaxPos.y = -1; } else { @@ -1893,8 +1893,8 @@ } } - *HitTest = co_IntSendMessage(ScopeWin->head.h, WM_NCHITTEST, 0, - MAKELONG(Point->x, Point->y)); + *HitTest = (USHORT)co_IntSendMessage(ScopeWin->head.h, WM_NCHITTEST, 0, + MAKELONG(Point->x, Point->y)); if ((*HitTest) == (USHORT)HTTRANSPARENT) { UserDereferenceObject(ScopeWin); @@ -2046,7 +2046,7 @@ TRACE("hwnd %p, after %p, %d,%d (%dx%d), flags %08x\n", winpos->pos.hwnd, winpos->pos.hwndInsertAfter, winpos->pos.x, winpos->pos.y, winpos->pos.cx, winpos->pos.cy, winpos->pos.flags); - + pwnd = UserGetWindowObject(winpos->pos.hwnd); if (!pwnd) continue; Modified: trunk/reactos/subsystems/win32/win32k/objects/arc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/arc.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/arc.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -126,10 +126,10 @@ AngleEnd = atan2((RectSEpts.bottom - CenterY), RectSEpts.right - CenterX)*(360.0/(M_PI*2)); AngleStart = atan2((RectSEpts.top - CenterY), RectSEpts.left - CenterX)*(360.0/(M_PI*2)); - SfCx = (Rcos(AngleStart) * RadiusX); - SfCy = (Rsin(AngleStart) * RadiusY); - EfCx = (Rcos(AngleEnd) * RadiusX); - EfCy = (Rsin(AngleEnd) * RadiusY); + SfCx = (LONG)(Rcos(AngleStart) * RadiusX); + SfCy = (LONG)(Rsin(AngleStart) * RadiusY); + EfCx = (LONG)(Rcos(AngleEnd) * RadiusX); + EfCy = (LONG)(Rsin(AngleEnd) * RadiusY); if ((arctype == GdiTypePie) || (arctype == GdiTypeChord)) { Modified: trunk/reactos/subsystems/win32/win32k/objects/bitmaps.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/bitmaps.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/bitmaps.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -158,11 +158,11 @@ /* Calculate width and image size in bytes */ cjWidthBytes = WIDTH_BYTES_ALIGN16(nWidth, cRealBpp); - cjSize = cjWidthBytes * nHeight; - - /* Check parameters (possible overflow of cjWidthBytes!) */ - if (iFormat == 0 || nWidth <= 0 || nWidth >= 0x8000000 || nHeight <= 0 || - cBitsPixel > 32 || cPlanes > 32 || cjSize >= 0x100000000ULL) + cjSize = (ULONGLONG)cjWidthBytes * nHeight; + + /* Check parameters (possible overflow of cjSize!) */ + if ((iFormat == 0) || (nWidth <= 0) || (nWidth >= 0x8000000) || (nHeight <= 0) || + (cBitsPixel > 32) || (cPlanes > 32) || (cjSize >= 0x100000000ULL)) { DPRINT1("Invalid bitmap format! Width=%d, Height=%d, Bpp=%d, Planes=%d\n", nWidth, nHeight, cBitsPixel, cPlanes); @@ -178,7 +178,7 @@ PSURFACE psurf = SURFACE_ShareLockSurface(hbmp); _SEH2_TRY { - ProbeForRead(pUnsafeBits, cjSize, 1); + ProbeForRead(pUnsafeBits, (SIZE_T)cjSize, 1); UnsafeSetBitmapBits(psurf, 0, pUnsafeBits); } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) Modified: trunk/reactos/subsystems/win32/win32k/objects/dibobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/dibobj.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/dibobj.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -141,9 +141,9 @@ } biBitCount = BitsPerFormat(psurf->SurfObj.iBitmapFormat); - if (biBitCount <= 8 && StartIndex < (1 << biBitCount)) - { - if (StartIndex + Entries > (1 << biBitCount)) + if ((biBitCount <= 8) && (StartIndex < (1UL << biBitCount))) + { + if (StartIndex + Entries > (1UL << biBitCount)) Entries = (1 << biBitCount) - StartIndex; if (psurf->ppal == NULL) @@ -537,7 +537,8 @@ LONG width, height; WORD planes, bpp; DWORD compr, size ; - int i, bitmap_type; + USHORT i; + int bitmap_type; RGBTRIPLE* rgbTriples; RGBQUAD* rgbQuads; VOID* colorPtr; @@ -618,10 +619,10 @@ case 0: /* Only info */ if(pbmci) { - pbmci->bmciHeader.bcWidth = psurf->SurfObj.sizlBitmap.cx; - pbmci->bmciHeader.bcHeight = (psurf->SurfObj.fjBitmap & BMF_TOPDOWN) ? + pbmci->bmciHeader.bcWidth = (WORD)psurf->SurfObj.sizlBitmap.cx; + pbmci->bmciHeader.bcHeight = (WORD)((psurf->SurfObj.fjBitmap & BMF_TOPDOWN) ? -psurf->SurfObj.sizlBitmap.cy : - psurf->SurfObj.sizlBitmap.cy; + psurf->SurfObj.sizlBitmap.cy); pbmci->bmciHeader.bcPlanes = 1; pbmci->bmciHeader.bcBitCount = BitsPerFormat(psurf->SurfObj.iBitmapFormat); } @@ -674,19 +675,19 @@ { if(Usage == DIB_RGB_COLORS) { - unsigned int colors = min(psurf->ppal->NumColors, 1 << bpp); + ULONG colors = min(psurf->ppal->NumColors, 256); if(pbmci) { - for(i=0; i < colors; i++) + for(i = 0; i < colors; i++) { rgbTriples[i].rgbtRed = psurf->ppal->IndexedColors[i].peRed; rgbTriples[i].rgbtGreen = psurf->ppal->IndexedColors[i].peGreen; rgbTriples[i].rgbtBlue = psurf->ppal->IndexedColors[i].peBlue; } } - if(colors != 1 << bpp) Info->bmiHeader.biClrUsed = colors; - for(i=0; i < colors; i++) + if(colors != 256) Info->bmiHeader.biClrUsed = colors; + for(i = 0; i < colors; i++) { rgbQuads[i].rgbRed = psurf->ppal->IndexedColors[i].peRed; rgbQuads[i].rgbGreen = psurf->ppal->IndexedColors[i].peGreen; @@ -695,7 +696,7 @@ } else { - for(i=0; i < 1 << bpp; i++) + for(i = 0; i < 256; i++) { if(pbmci) ((WORD*)rgbTriples)[i] = i; ((WORD*)rgbQuads)[i] = i; @@ -706,7 +707,7 @@ { if(Usage == DIB_PAL_COLORS) { - for(i=0; i < 1 << bpp; i++) + for(i = 0; i < 256; i++) { if(pbmci) ((WORD*)rgbTriples)[i] = i; ((WORD*)rgbQuads)[i] = i; @@ -877,7 +878,7 @@ POINTL srcPoint; BOOL ret ; - if (StartScan > psurf->SurfObj.sizlBitmap.cy) + if (StartScan > (ULONG)psurf->SurfObj.sizlBitmap.cy) { ScanLines = 0; goto done; @@ -889,8 +890,8 @@ /* Fixup values */ Info->bmiHeader.biWidth = psurf->SurfObj.sizlBitmap.cx; - Info->bmiHeader.biHeight = height < 0 ? - -ScanLines : ScanLines; + Info->bmiHeader.biHeight = (height < 0) ? + -(LONG)ScanLines : ScanLines; /* Create the DIB */ hBmpDest = DIB_CreateDIBSection(pDC, Info, Usage, &pDIBits, NULL, 0, 0); /* Restore them */ @@ -1379,7 +1380,7 @@ { ProbeForRead(&bmi->bmiHeader.biSize, sizeof(DWORD), 1); ProbeForRead(bmi, bmi->bmiHeader.biSize, 1); - ProbeForRead(bmi, DIB_BitmapInfoSize(bmi, Usage), 1); + ProbeForRead(bmi, DIB_BitmapInfoSize(bmi, (WORD)Usage), 1); } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { @@ -1783,7 +1784,7 @@ FASTCALL BuildDIBPalette(CONST BITMAPINFO *bmi) { - BYTE bits; + WORD bits; ULONG ColorCount; HPALETTE hPal; ULONG RedMask = 0, GreenMask = 0, BlueMask = 0; Modified: trunk/reactos/subsystems/win32/win32k/objects/drawing.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/drawing.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/drawing.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -1315,14 +1315,14 @@ INT YLeft, INT Width, INT Height, - double StartArc, + double StartArc, // FIXME: don't use floating point! double EndArc, ARCTYPE arctype) { PDC_ATTR pdcattr; PBRUSH pbrush; - int Start = ceil(StartArc); - int End = ceil(EndArc); + int Start = (int)ceil(StartArc); + int End = (int)ceil(EndArc); BOOL Chord = (arctype == GdiTypeChord), ret; pdcattr = dc->pdcattr; @@ -1351,13 +1351,13 @@ INT YLeft, INT Width, INT Height, - double StartArc, + double StartArc, // FIXME: don't use floating point! double EndArc, ARCTYPE arctype, PBRUSH pbrush) { - int Start = ceil(StartArc); - int End = ceil(EndArc); + int Start = (int)ceil(StartArc); + int End = (int)ceil(EndArc); BOOL Chord = (arctype == GdiTypeChord); // Sort out alignment here. return app_draw_arc(dc, rect( XLeft, YLeft, Width, Height), Modified: trunk/reactos/subsystems/win32/win32k/objects/fillshap.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/fillshap.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/fillshap.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -355,7 +355,7 @@ PULONG SafeCounts; NTSTATUS Status = STATUS_SUCCESS; BOOL Ret = TRUE; - INT nPoints = 0, nMaxPoints = 0, nInvalid = 0, i; + ULONG nPoints = 0, nMaxPoints = 0, nInvalid = 0, i; if (!UnsafePoints || !UnsafeCounts || Count == 0 || iFunc == 0 || iFunc > GdiPolyPolyRgn) Modified: trunk/reactos/subsystems/win32/win32k/objects/gdiobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/gdiobj.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/gdiobj.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -1227,7 +1227,7 @@ { POBJ pobj; FLONG fl = 0; - UCHAR objt = ObjectType >> 16; + UCHAR objt = (ObjectType >> 16) & 0xFF; if ((objt == GDIObjType_DC_TYPE && cjSize == sizeof(DC)) || (objt == GDIObjType_PAL_TYPE && cjSize == sizeof(PALETTE)) || Modified: trunk/reactos/subsystems/win32/win32k/objects/line.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/line.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/line.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -334,7 +334,7 @@ PULONG PolyPoints, DWORD Count) { - int i; + ULONG i; LPPOINT pts; PULONG pc; BOOL ret = FALSE; // Default to failure @@ -419,7 +419,7 @@ PDC dc; PDC_ATTR pdcattr; POINT *line_pts = NULL, *line_pts_old, *bzr_pts = NULL, bzr[4]; - INT i, num_pts, num_bzr_pts, space, space_old, size; + ULONG i, num_pts, num_bzr_pts, space, space_old, size; BOOL result = FALSE; dc = DC_LockDc(hdc); Modified: trunk/reactos/subsystems/win32/win32k/objects/palette.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/palette.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/palette.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -404,7 +404,7 @@ if (ppdev->flFlags & PDEV_GAMMARAMP_TABLE) { - INT i; + ULONG i; PGAMMARAMP GammaRamp = (PGAMMARAMP)ppdev->pvGammaRamp; for ( i = 0; i < Colors; i++) { @@ -980,7 +980,7 @@ CONST LPPALETTEENTRY pe) { PPALETTE palGDI; - WORD numEntries; + ULONG numEntries; if ((UINT)hpal & GDI_HANDLE_STOCK_MASK) { Modified: trunk/reactos/subsystems/win32/win32k/objects/path.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/path.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/path.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -13,6 +13,10 @@ #define NDEBUG #include <debug.h> +#ifdef _MSC_VER +#pragma warning(disable:4244) +#endif + #define NUM_ENTRIES_INITIAL 16 /* Initial size of points / flags arrays */ #define GROW_FACTOR_NUMER 2 /* Numerator of grow factor for the array */ #define GROW_FACTOR_DENOM 1 /* Denominator of grow factor */ @@ -20,7 +24,7 @@ /*********************************************************************** * Internal functions */ - + /* PATH_DestroyGdiPath * * Destroys a GdiPath structure (frees the memory in the arrays). @@ -71,7 +75,7 @@ GdiPathDPtoLP(PDC pdc, PPOINT ppt, INT count) { XFORMOBJ xo; - + XFORMOBJ_vInit(&xo, &pdc->dclevel.mxDeviceToWorld); return XFORMOBJ_bApplyXform(&xo, XF_LTOL, count, (PPOINTL)ppt, (PPOINTL)ppt); } @@ -738,7 +742,7 @@ { PPATH pPath; POINT lastmove, orig_pos; - INT i; + ULONG i; PDC_ATTR pdcattr; BOOL State = FALSE, Ret = FALSE; @@ -751,7 +755,7 @@ return FALSE; } - pdcattr = dc->pdcattr; + pdcattr = dc->pdcattr; lastmove.x = orig_pos.x = pdcattr->ptlCurrent.x; lastmove.y = orig_pos.y = pdcattr->ptlCurrent.y; Modified: trunk/reactos/subsystems/win32/win32k/objects/pen.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/win32k/ob… ============================================================================== --- trunk/reactos/subsystems/win32/win32k/objects/pen.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/win32k/objects/pen.c [iso-8859-1] Sun Mar 4 11:32:43 2012 @@ -3,7 +3,7 @@ * PROJECT: ReactOS kernel * PURPOSE: Pen functiona * FILE: subsys/win32k/objects/pen.c - * PROGRAMER: + * PROGRAMER: */ #include <win32k.h> @@ -230,7 +230,7 @@ cbRetCount = sizeof(EXTLOGPEN) - sizeof(DWORD) + pbrushPen->dwStyleCount * sizeof(DWORD); if (pBuffer) { - INT i; + ULONG i; if (cbCount < cbRetCount) return 0; pExtLogPen = (PEXTLOGPEN)pBuffer;

12 years, 8 months

[janderwald] 55992: [LIBUSB] - Fix bug while scanning endpoint descriptors - Do not assume interface info has the correct number of pipes set - Fix alternate interface handling - Tested in VBox + U...

by janderwald＠svn.reactos.org

Author: janderwald Date: Sun Mar 4 11:07:13 2012 New Revision: 55992 URL: http://svn.reactos.org/svn/reactos?rev=55992&view=rev Log: [LIBUSB] - Fix bug while scanning endpoint descriptors - Do not assume interface info has the correct number of pipes set - Fix alternate interface handling - Tested in VBox + USB Audio Device Modified: trunk/reactos/lib/drivers/libusb/hub_controller.cpp trunk/reactos/lib/drivers/libusb/libusb.h trunk/reactos/lib/drivers/libusb/usb_device.cpp Modified: trunk/reactos/lib/drivers/libusb/hub_controller.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/libusb/hub_con… ============================================================================== --- trunk/reactos/lib/drivers/libusb/hub_controller.cpp [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/libusb/hub_controller.cpp [iso-8859-1] Sun Mar 4 11:07:13 2012 @@ -798,6 +798,7 @@ // sanity checks // ASSERT(EndPointDesc); + DPRINT("HandleIsochronousTransfer EndPointDesc %p Address %x bmAttributes %x\n", EndPointDesc, EndPointDesc->bEndpointAddress, EndPointDesc->bmAttributes); ASSERT((EndPointDesc->bmAttributes & USB_ENDPOINT_TYPE_MASK) == USB_ENDPOINT_TYPE_ISOCHRONOUS); // Modified: trunk/reactos/lib/drivers/libusb/libusb.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/libusb/libusb.… ============================================================================== --- trunk/reactos/lib/drivers/libusb/libusb.h [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/libusb/libusb.h [iso-8859-1] Sun Mar 4 11:07:13 2012 @@ -12,7 +12,6 @@ { #include <usbdlib.h> } - // // FIXME: Modified: trunk/reactos/lib/drivers/libusb/usb_device.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/drivers/libusb/usb_dev… ============================================================================== --- trunk/reactos/lib/drivers/libusb/usb_device.cpp [iso-8859-1] (original) +++ trunk/reactos/lib/drivers/libusb/usb_device.cpp [iso-8859-1] Sun Mar 4 11:07:13 2012 @@ -65,6 +65,8 @@ virtual VOID DumpDeviceDescriptor(PUSB_DEVICE_DESCRIPTOR DeviceDescriptor); virtual VOID DumpConfigurationDescriptor(PUSB_CONFIGURATION_DESCRIPTOR ConfigurationDescriptor); virtual NTSTATUS GetConfigurationDescriptor(UCHAR ConfigurationIndex, USHORT BufferSize, PVOID Buffer); + virtual NTSTATUS BuildInterfaceDescriptor(IN ULONG ConfigurationIndex, IN PUSB_INTERFACE_DESCRIPTOR InterfaceDescriptor, OUT PUSBD_INTERFACE_INFORMATION InterfaceInfo, OUT PUSB_INTERFACE *OutUsbInterface); + // constructor / destructor CUSBDevice(IUnknown *OuterUnknown){} @@ -916,6 +918,92 @@ // return Status; } +//---------------------------------------------------------------------------------------- +NTSTATUS +CUSBDevice::BuildInterfaceDescriptor( + IN ULONG ConfigurationIndex, + IN PUSB_INTERFACE_DESCRIPTOR InterfaceDescriptor, + OUT PUSBD_INTERFACE_INFORMATION InterfaceInfo, + OUT PUSB_INTERFACE *OutUsbInterface) +{ + PUSB_INTERFACE UsbInterface; + PUSB_ENDPOINT_DESCRIPTOR EndpointDescriptor; + ULONG PipeIndex; + + // allocate interface handle + UsbInterface = (PUSB_INTERFACE)ExAllocatePool(NonPagedPool, sizeof(USB_INTERFACE) + (InterfaceDescriptor->bNumEndpoints - 1) * sizeof(USB_ENDPOINT)); + if (!UsbInterface) + { + // failed to allocate memory + return STATUS_INSUFFICIENT_RESOURCES; + } + + // zero descriptor + RtlZeroMemory(UsbInterface, sizeof(USB_INTERFACE) + (InterfaceDescriptor->bNumEndpoints - 1) * sizeof(USB_ENDPOINT)); + + // store handle + InterfaceInfo->InterfaceHandle = (USBD_INTERFACE_HANDLE)UsbInterface; + InterfaceInfo->Class = InterfaceDescriptor->bInterfaceClass; + InterfaceInfo->SubClass = InterfaceDescriptor->bInterfaceSubClass; + InterfaceInfo->Protocol = InterfaceDescriptor->bInterfaceProtocol; + InterfaceInfo->Reserved = 0; + + + // init interface handle + UsbInterface->InterfaceDescriptor = InterfaceDescriptor; + InsertTailList(&m_ConfigurationDescriptors[ConfigurationIndex].InterfaceList, &UsbInterface->ListEntry); + + // grab first endpoint descriptor + EndpointDescriptor = (PUSB_ENDPOINT_DESCRIPTOR) (InterfaceDescriptor + 1); + + // now copy all endpoint information + for(PipeIndex = 0; PipeIndex < InterfaceDescriptor->bNumEndpoints; PipeIndex++) + { + while(EndpointDescriptor->bDescriptorType != USB_ENDPOINT_DESCRIPTOR_TYPE) + { + // skip intermediate descriptors + if (EndpointDescriptor->bLength == 0 || EndpointDescriptor->bDescriptorType == USB_INTERFACE_DESCRIPTOR_TYPE) + { + // bogus configuration descriptor + DPRINT1("[USBEHCI] Bogus descriptor found in InterfaceNumber %x Alternate %x EndpointIndex %x bLength %x bDescriptorType %x\n", InterfaceDescriptor->bInterfaceNumber, InterfaceDescriptor->bAlternateSetting, PipeIndex, + EndpointDescriptor->bLength, EndpointDescriptor->bDescriptorType); + + // failed + return STATUS_UNSUCCESSFUL; + } + + // move to next descriptor + EndpointDescriptor = (PUSB_ENDPOINT_DESCRIPTOR)((ULONG_PTR)EndpointDescriptor + EndpointDescriptor->bLength); + } + + // store in interface info + RtlCopyMemory(&UsbInterface->EndPoints[PipeIndex].EndPointDescriptor, EndpointDescriptor, sizeof(USB_ENDPOINT_DESCRIPTOR)); + + DPRINT("Configuration Descriptor %p Length %lu\n", m_ConfigurationDescriptors[ConfigurationIndex].ConfigurationDescriptor, m_ConfigurationDescriptors[ConfigurationIndex].ConfigurationDescriptor->wTotalLength); + DPRINT("EndpointDescriptor %p DescriptorType %x bLength %x\n", EndpointDescriptor, EndpointDescriptor->bDescriptorType, EndpointDescriptor->bLength); + DPRINT("EndpointDescriptorHandle %p bAddress %x bmAttributes %x\n",&UsbInterface->EndPoints[PipeIndex], UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bEndpointAddress, + UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bmAttributes); + + // copy pipe info + InterfaceInfo->Pipes[PipeIndex].MaximumPacketSize = UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.wMaxPacketSize; + InterfaceInfo->Pipes[PipeIndex].EndpointAddress = UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bEndpointAddress; + InterfaceInfo->Pipes[PipeIndex].Interval = UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bInterval; + InterfaceInfo->Pipes[PipeIndex].PipeType = (USBD_PIPE_TYPE)UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bmAttributes; + InterfaceInfo->Pipes[PipeIndex].PipeHandle = (PVOID)&UsbInterface->EndPoints[PipeIndex]; + + // move to next descriptor + EndpointDescriptor = (PUSB_ENDPOINT_DESCRIPTOR)((ULONG_PTR)EndpointDescriptor + EndpointDescriptor->bLength); + } + + if (OutUsbInterface) + { + // output result + *OutUsbInterface = UsbInterface; + } + return STATUS_SUCCESS; + +} + //---------------------------------------------------------------------------------------- NTSTATUS @@ -924,7 +1012,7 @@ IN PUSBD_INTERFACE_INFORMATION InterfaceInfo, OUT USBD_CONFIGURATION_HANDLE *ConfigurationHandle) { - ULONG InterfaceIndex, PipeIndex; + ULONG InterfaceIndex; USB_DEFAULT_PIPE_SETUP_PACKET CtrlSetup; NTSTATUS Status; UCHAR bConfigurationValue = 0; @@ -932,7 +1020,6 @@ UCHAR Found = FALSE; PUSB_INTERFACE_DESCRIPTOR InterfaceDescriptor; PUSB_INTERFACE UsbInterface; - PUSB_ENDPOINT_DESCRIPTOR EndpointDescriptor; PLIST_ENTRY Entry; if (ConfigurationDescriptor) @@ -1029,61 +1116,11 @@ ASSERT(InterfaceInfo->NumberOfPipes == InterfaceDescriptor->bNumEndpoints); // copy interface info - InterfaceInfo->Class = InterfaceDescriptor->bInterfaceClass; - InterfaceInfo->SubClass = InterfaceDescriptor->bInterfaceSubClass; - InterfaceInfo->Protocol = InterfaceDescriptor->bInterfaceProtocol; - InterfaceInfo->Reserved = 0; - - // allocate interface handle - UsbInterface = (PUSB_INTERFACE)ExAllocatePool(NonPagedPool, sizeof(USB_INTERFACE) + (InterfaceInfo->NumberOfPipes - 1) * sizeof(USB_ENDPOINT)); - if (!UsbInterface) + Status = BuildInterfaceDescriptor(ConfigurationIndex, InterfaceDescriptor, InterfaceInfo, NULL); + if (!NT_SUCCESS(Status)) { - // failed to allocate memory - return STATUS_INSUFFICIENT_RESOURCES; - } - - // store handle - InterfaceInfo->InterfaceHandle = (USBD_INTERFACE_HANDLE)UsbInterface; - - // init interface handle - UsbInterface->InterfaceDescriptor = InterfaceDescriptor; - InsertTailList(&m_ConfigurationDescriptors[ConfigurationIndex].InterfaceList, &UsbInterface->ListEntry); - - // grab first endpoint descriptor - EndpointDescriptor = (PUSB_ENDPOINT_DESCRIPTOR) (InterfaceDescriptor + 1); - - // now copy all endpoint information - for(PipeIndex = 0; PipeIndex < InterfaceInfo->NumberOfPipes; PipeIndex++) - { - while(EndpointDescriptor->bDescriptorType != USB_ENDPOINT_DESCRIPTOR_TYPE && EndpointDescriptor->bLength != sizeof(USB_ENDPOINT_DESCRIPTOR)) - { - // skip intermediate descriptors - if (EndpointDescriptor->bLength == 0 || EndpointDescriptor->bDescriptorType == USB_INTERFACE_DESCRIPTOR_TYPE) - { - // bogus configuration descriptor - DPRINT1("[USBEHCI] Bogus descriptor found in InterfaceNumber %x Alternate %x EndpointIndex %x bLength %x bDescriptorType %x\n", InterfaceDescriptor->bInterfaceNumber, InterfaceDescriptor->bAlternateSetting, PipeIndex, - EndpointDescriptor->bLength, EndpointDescriptor->bDescriptorType); - - // failed - return STATUS_UNSUCCESSFUL; - } - - // move to next descriptor - EndpointDescriptor = (PUSB_ENDPOINT_DESCRIPTOR)((ULONG_PTR)EndpointDescriptor + EndpointDescriptor->bLength); - } - - // store in interface info - RtlCopyMemory(&UsbInterface->EndPoints[PipeIndex].EndPointDescriptor, EndpointDescriptor, sizeof(USB_ENDPOINT_DESCRIPTOR)); - - // copy pipe info - InterfaceInfo->Pipes[PipeIndex].MaximumPacketSize = UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.wMaxPacketSize; - InterfaceInfo->Pipes[PipeIndex].EndpointAddress = UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bEndpointAddress; - InterfaceInfo->Pipes[PipeIndex].Interval = UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bInterval; - InterfaceInfo->Pipes[PipeIndex].PipeType = (USBD_PIPE_TYPE)UsbInterface->EndPoints[PipeIndex].EndPointDescriptor.bmAttributes; - InterfaceInfo->Pipes[PipeIndex].PipeHandle = (PVOID)&UsbInterface->EndPoints[PipeIndex]; - - // move to next descriptor - EndpointDescriptor = (PUSB_ENDPOINT_DESCRIPTOR)((ULONG_PTR)EndpointDescriptor + EndpointDescriptor->bLength); + // failed + break; } // move offset @@ -1108,6 +1145,7 @@ ULONG Index, ConfigurationIndex = 0, Found = FALSE; PUSB_INTERFACE UsbInterface; PLIST_ENTRY Entry; + PUSB_INTERFACE_DESCRIPTOR InterfaceDescriptor; // check if handle is valid for(Index = 0; Index < m_DeviceDescriptor.bNumConfigurations; Index++) @@ -1171,17 +1209,35 @@ if (!Found) { - // selected interface but interface not found - // something is really wrong - DPRINT1("[USBEHCI] Error: interface not found!!!\n"); - return STATUS_UNSUCCESSFUL; + // find interface descriptor + InterfaceDescriptor = USBD_ParseConfigurationDescriptor(m_ConfigurationDescriptors[ConfigurationIndex].ConfigurationDescriptor, InterfaceInfo->InterfaceNumber, InterfaceInfo->AlternateSetting); + if (!InterfaceDescriptor) + { + DPRINT1("[LIBUSB] No such interface Alternate %x InterfaceNumber %x\n", InterfaceInfo->AlternateSetting, InterfaceInfo->InterfaceNumber); + return STATUS_UNSUCCESSFUL; + } + + // build interface descriptor + Status = BuildInterfaceDescriptor(ConfigurationIndex, InterfaceDescriptor, InterfaceInfo, &UsbInterface); + if (!NT_SUCCESS(Status)) + { + // failed + DPRINT1("[LIBUSB] Failed to build interface descriptor Status %x\n", Status); + return Status; + } } // assert on pipe length mismatch - ASSERT(InterfaceInfo->NumberOfPipes == UsbInterface->InterfaceDescriptor->bNumEndpoints); + DPRINT1("NumberOfPipes %lu Endpoints %lu Length %lu\n", InterfaceInfo->NumberOfPipes, UsbInterface->InterfaceDescriptor->bNumEndpoints, InterfaceInfo->Length); + + // sanity check + ASSERT(GET_USBD_INTERFACE_SIZE(UsbInterface->InterfaceDescriptor->bNumEndpoints) == InterfaceInfo->Length); + + // store number of pipes + InterfaceInfo->NumberOfPipes = UsbInterface->InterfaceDescriptor->bNumEndpoints; // copy pipe handles - for(PipeIndex = 0; PipeIndex < min(InterfaceInfo->NumberOfPipes, UsbInterface->InterfaceDescriptor->bNumEndpoints); PipeIndex++) + for(PipeIndex = 0; PipeIndex < UsbInterface->InterfaceDescriptor->bNumEndpoints; PipeIndex++) { // copy pipe handle DPRINT1("PipeIndex %lu\n", PipeIndex);

12 years, 8 months

[sir_richard] 55991: [NTOS]: If the address space is exhausted, don't ASSERT, simply return failure just as Windows does. Should fix the last of the ASSERTions on TestBot. Next step is to support V...

by sir_richard＠svn.reactos.org

Author: sir_richard Date: Sun Mar 4 07:04:42 2012 New Revision: 55991 URL: http://svn.reactos.org/svn/reactos?rev=55991&view=rev Log: [NTOS]: If the address space is exhausted, don't ASSERT, simply return failure just as Windows does. Should fix the last of the ASSERTions on TestBot. Next step is to support VirtualProtect/Query correctly. Modified: trunk/reactos/ntoskrnl/mm/ARM3/virtual.c Modified: trunk/reactos/ntoskrnl/mm/ARM3/virtual.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/virtual.c… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] Sun Mar 4 07:04:42 2012 @@ -3345,7 +3345,7 @@ &Process->VadRoot, (PMMADDRESS_NODE*)&Process->VadFreeHint, &StartingAddress); - ASSERT(NT_SUCCESS(Status)); + if (!NT_SUCCESS(Status)) goto FailPath; // // Now we know where the allocation ends. Make sure it doesn't end up

12 years, 8 months

[sir_richard] 55990: [NTOS]: Blimey this was a hard one. When using the reserved flag to request 1MB in a new process (which is used for starting SMSS and CSRSS), we must request 1MB - 256 bytes (o...

by sir_richard＠svn.reactos.org

Author: sir_richard Date: Sun Mar 4 06:42:49 2012 New Revision: 55990 URL: http://svn.reactos.org/svn/reactos?rev=55990&view=rev Log: [NTOS]: Blimey this was a hard one. When using the reserved flag to request 1MB in a new process (which is used for starting SMSS and CSRSS), we must request 1MB - 256 bytes (or any number, actually) to offset the fact that with a base address of 0x4, a 1MB region gets us at 0x100FFF, and not 0xFFFF, because Windows computes ending address *before* alignment, and then returns a new region size for you (so if you pass in 1MB, you get 1MB + 4096KB). In Win32csr, when the code is trying to release 1MB, this ended up in our "Case A", because it would still leave a page in the VAD. Fixed rtl to request just shy off a MB. Verified on Windows as well. [NTOS]: The fix above was due to fixing "EndingAddress" which was being initialized to zero too late (after writing to it!). This caused allocations with a fixed base address that were already on top of another allocation not to be seen as a conflict, then we tried inserting a VAD and received an ASSERT saying we've already found a VAD there. After fixing the sizing code, the bug above creeped up. Whoever wrote the NtFreeVirtualMemory test is a godsend. It has been nailing bug after bug in the VAD implementation. Thank you. Modified: trunk/reactos/lib/rtl/process.c trunk/reactos/ntoskrnl/mm/ARM3/virtual.c Modified: trunk/reactos/lib/rtl/process.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/process.c?rev=5599… ============================================================================== --- trunk/reactos/lib/rtl/process.c [iso-8859-1] (original) +++ trunk/reactos/lib/rtl/process.c [iso-8859-1] Sun Mar 4 06:42:49 2012 @@ -84,7 +84,7 @@ { /* Give 1MB starting at 0x4 */ BaseAddress = (PVOID)4; - EnviroSize = 1024 * 1024; + EnviroSize = (1024 * 1024) - 256; Status = ZwAllocateVirtualMemory(ProcessHandle, &BaseAddress, 0, Modified: trunk/reactos/ntoskrnl/mm/ARM3/virtual.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/virtual.c… ============================================================================== --- trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] Sun Mar 4 06:42:49 2012 @@ -3295,6 +3295,8 @@ // PRegionSize = ROUND_TO_PAGES(PRegionSize); PageCount = BYTES_TO_PAGES(PRegionSize); + EndingAddress = 0; + StartingAddress = 0; } else { @@ -3336,7 +3338,6 @@ // which was passed in isn't already conflicting with an existing address // range. // - EndingAddress = 0; if (!PBaseAddress) { Status = MiFindEmptyAddressRangeInTree(PRegionSize, @@ -3345,6 +3346,17 @@ (PMMADDRESS_NODE*)&Process->VadFreeHint, &StartingAddress); ASSERT(NT_SUCCESS(Status)); + + // + // Now we know where the allocation ends. Make sure it doesn't end up + // somewhere in kernel mode. + // + EndingAddress = ((ULONG_PTR)StartingAddress + PRegionSize - 1) | (PAGE_SIZE - 1); + if ((PVOID)EndingAddress > MM_HIGHEST_VAD_ADDRESS) + { + Status = STATUS_NO_MEMORY; + goto FailPath; + } } else if (MiCheckForConflictingNode(StartingAddress >> PAGE_SHIFT, EndingAddress >> PAGE_SHIFT, @@ -3354,17 +3366,6 @@ // The address specified is in conflict! // Status = STATUS_CONFLICTING_ADDRESSES; - goto FailPath; - } - - // - // Now we know where the allocation ends. Make sure it doesn't end up - // somewhere in kernel mode. - // - EndingAddress = ((ULONG_PTR)StartingAddress + PRegionSize - 1) | (PAGE_SIZE - 1); - if ((PVOID)EndingAddress > MM_HIGHEST_VAD_ADDRESS) - { - Status = STATUS_NO_MEMORY; goto FailPath; }

12 years, 8 months

← Newer
1
...
27
28
29
30
31
32
33
...
36
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs March 2012