Ros-diffs December 2017

ros-diffs@reactos.org

26 participants
355 discussions

[reactos] 01/01: [MSPAINT] Add some header guards.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4db1f987f3e113c871d98… commit 4db1f987f3e113c871d98a34b80ed4b3d643332a Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 13:36:45 2017 +0100 [MSPAINT] Add some header guards. --- base/applications/mspaint/definitions.h | 2 ++ base/applications/mspaint/dialogs.h | 2 ++ base/applications/mspaint/dib.h | 2 ++ base/applications/mspaint/drawing.h | 2 ++ base/applications/mspaint/fullscreen.h | 2 ++ base/applications/mspaint/globalvar.h | 2 ++ base/applications/mspaint/history.h | 2 ++ base/applications/mspaint/imgarea.h | 2 ++ base/applications/mspaint/miniature.h | 2 ++ base/applications/mspaint/mouse.h | 2 ++ base/applications/mspaint/palette.h | 2 ++ base/applications/mspaint/palettemodel.h | 2 ++ base/applications/mspaint/registry.h | 2 ++ base/applications/mspaint/scrollbox.h | 2 ++ base/applications/mspaint/selection.h | 2 ++ base/applications/mspaint/selectionmodel.h | 2 ++ base/applications/mspaint/sizebox.h | 2 ++ base/applications/mspaint/textedit.h | 2 ++ base/applications/mspaint/toolbox.h | 2 ++ base/applications/mspaint/toolsettings.h | 2 ++ base/applications/mspaint/toolsmodel.h | 2 ++ base/applications/mspaint/winproc.h | 2 ++ 22 files changed, 44 insertions(+) diff --git a/base/applications/mspaint/definitions.h b/base/applications/mspaint/definitions.h index 818cc5c25f..6d5e7bc109 100644 --- a/base/applications/mspaint/definitions.h +++ b/base/applications/mspaint/definitions.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + /* DEFINES **********************************************************/ #define HISTORYSIZE 11 diff --git a/base/applications/mspaint/dialogs.h b/base/applications/mspaint/dialogs.h index 7c85652601..701c23f20e 100644 --- a/base/applications/mspaint/dialogs.h +++ b/base/applications/mspaint/dialogs.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CMirrorRotateDialog : public CDialogImpl<CMirrorRotateDialog> { public: diff --git a/base/applications/mspaint/dib.h b/base/applications/mspaint/dib.h index 7a25f2b9d0..743c2e8e6a 100644 --- a/base/applications/mspaint/dib.h +++ b/base/applications/mspaint/dib.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + HBITMAP CreateDIBWithProperties(int width, int height); int GetDIBWidth(HBITMAP hbm); diff --git a/base/applications/mspaint/drawing.h b/base/applications/mspaint/drawing.h index 9cc5c7886f..737d035250 100644 --- a/base/applications/mspaint/drawing.h +++ b/base/applications/mspaint/drawing.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + void Line(HDC hdc, LONG x1, LONG y1, LONG x2, LONG y2, COLORREF color, int thickness); void Rect(HDC hdc, LONG x1, LONG y1, LONG x2, LONG y2, COLORREF fg, COLORREF bg, int thickness, int style); diff --git a/base/applications/mspaint/fullscreen.h b/base/applications/mspaint/fullscreen.h index 4ff4779290..2a41d09978 100644 --- a/base/applications/mspaint/fullscreen.h +++ b/base/applications/mspaint/fullscreen.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CFullscreenWindow : public CWindowImpl<CFullscreenWindow> { public: diff --git a/base/applications/mspaint/globalvar.h b/base/applications/mspaint/globalvar.h index d584eb6375..ab42977518 100644 --- a/base/applications/mspaint/globalvar.h +++ b/base/applications/mspaint/globalvar.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + /* VARIABLES declared in main.cpp ***********************************/ class RegistrySettings; diff --git a/base/applications/mspaint/history.h b/base/applications/mspaint/history.h index c87ff60f7b..7ed0409cfe 100644 --- a/base/applications/mspaint/history.h +++ b/base/applications/mspaint/history.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class ImageModel { private: diff --git a/base/applications/mspaint/imgarea.h b/base/applications/mspaint/imgarea.h index 79cc13ddf5..2fd0074b25 100644 --- a/base/applications/mspaint/imgarea.h +++ b/base/applications/mspaint/imgarea.h @@ -8,6 +8,8 @@ * Katayama Hirofumi MZ */ +#pragma once + class CImgAreaWindow : public CWindowImpl<CMainWindow> { public: diff --git a/base/applications/mspaint/miniature.h b/base/applications/mspaint/miniature.h index 44d81f4987..ce79455a23 100644 --- a/base/applications/mspaint/miniature.h +++ b/base/applications/mspaint/miniature.h @@ -7,6 +7,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CMiniatureWindow : public CWindowImpl<CMiniatureWindow> { public: diff --git a/base/applications/mspaint/mouse.h b/base/applications/mspaint/mouse.h index e26678165b..59f13fd742 100644 --- a/base/applications/mspaint/mouse.h +++ b/base/applications/mspaint/mouse.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + void placeSelWin(void); void startPaintingL(HDC hdc, LONG x, LONG y, COLORREF fg, COLORREF bg); diff --git a/base/applications/mspaint/palette.h b/base/applications/mspaint/palette.h index 3cb0e0784b..ca84fc5e9d 100644 --- a/base/applications/mspaint/palette.h +++ b/base/applications/mspaint/palette.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CPaletteWindow : public CWindowImpl<CPaletteWindow> { public: diff --git a/base/applications/mspaint/palettemodel.h b/base/applications/mspaint/palettemodel.h index 65ccb99252..04c86dbc66 100644 --- a/base/applications/mspaint/palettemodel.h +++ b/base/applications/mspaint/palettemodel.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + /* CLASSES **********************************************************/ class PaletteModel diff --git a/base/applications/mspaint/registry.h b/base/applications/mspaint/registry.h index 06beac503f..98cd407237 100644 --- a/base/applications/mspaint/registry.h +++ b/base/applications/mspaint/registry.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class RegistrySettings { private: diff --git a/base/applications/mspaint/scrollbox.h b/base/applications/mspaint/scrollbox.h index bad3bc7209..54c42b3d8a 100644 --- a/base/applications/mspaint/scrollbox.h +++ b/base/applications/mspaint/scrollbox.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CScrollboxWindow : public CWindowImpl<CScrollboxWindow> { public: diff --git a/base/applications/mspaint/selection.h b/base/applications/mspaint/selection.h index e2dacf2b14..536d4aa9a7 100644 --- a/base/applications/mspaint/selection.h +++ b/base/applications/mspaint/selection.h @@ -7,6 +7,8 @@ * Katayama Hirofumi MZ */ +#pragma once + class CSelectionWindow : public CWindowImpl<CSelectionWindow> { public: diff --git a/base/applications/mspaint/selectionmodel.h b/base/applications/mspaint/selectionmodel.h index d5ddf1f1d2..073cef7ea1 100644 --- a/base/applications/mspaint/selectionmodel.h +++ b/base/applications/mspaint/selectionmodel.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + /* DEFINES **********************************************************/ #define ACTION_MOVE 0 diff --git a/base/applications/mspaint/sizebox.h b/base/applications/mspaint/sizebox.h index afe89ba283..551f0e6bfb 100644 --- a/base/applications/mspaint/sizebox.h +++ b/base/applications/mspaint/sizebox.h @@ -7,6 +7,8 @@ * Katayama Hirofumi MZ */ +#pragma once + class CSizeboxWindow : public CWindowImpl<CSizeboxWindow> { public: diff --git a/base/applications/mspaint/textedit.h b/base/applications/mspaint/textedit.h index 4c44c2d7e6..61eac3675a 100644 --- a/base/applications/mspaint/textedit.h +++ b/base/applications/mspaint/textedit.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CTextEditWindow : public CWindowImpl<CTextEditWindow> { public: diff --git a/base/applications/mspaint/toolbox.h b/base/applications/mspaint/toolbox.h index 802b198ded..772ff2df1e 100644 --- a/base/applications/mspaint/toolbox.h +++ b/base/applications/mspaint/toolbox.h @@ -7,6 +7,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CToolBox : public CWindowImpl<CMainWindow> { public: diff --git a/base/applications/mspaint/toolsettings.h b/base/applications/mspaint/toolsettings.h index da8dc196ef..4b06cb7960 100644 --- a/base/applications/mspaint/toolsettings.h +++ b/base/applications/mspaint/toolsettings.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CToolSettingsWindow : public CWindowImpl<CToolSettingsWindow> { public: diff --git a/base/applications/mspaint/toolsmodel.h b/base/applications/mspaint/toolsmodel.h index 62855633c2..49ab71bcc3 100644 --- a/base/applications/mspaint/toolsmodel.h +++ b/base/applications/mspaint/toolsmodel.h @@ -6,6 +6,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + /* CLASSES **********************************************************/ class ToolsModel diff --git a/base/applications/mspaint/winproc.h b/base/applications/mspaint/winproc.h index 57da913b92..6b212f619b 100644 --- a/base/applications/mspaint/winproc.h +++ b/base/applications/mspaint/winproc.h @@ -7,6 +7,8 @@ * PROGRAMMERS: Benedikt Freisen */ +#pragma once + class CMainWindow : public CWindowImpl<CMainWindow> { public:

7 years

[reactos] 01/01: [LSASRV] Deduplicate SAMPR_ULONG_ARRAY.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=44d57334de6e0a2aefb06… commit 44d57334de6e0a2aefb06884f71f69288dc24702 Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 13:29:58 2017 +0100 [LSASRV] Deduplicate SAMPR_ULONG_ARRAY. --- dll/win32/lsasrv/authpackage.c | 6 ------ dll/win32/lsasrv/lookup.c | 7 ------- dll/win32/lsasrv/lsasrv.h | 6 ++++++ 3 files changed, 6 insertions(+), 13 deletions(-) diff --git a/dll/win32/lsasrv/authpackage.c b/dll/win32/lsasrv/authpackage.c index 7a3bf7ffc5..3cc43f7487 100644 --- a/dll/win32/lsasrv/authpackage.c +++ b/dll/win32/lsasrv/authpackage.c @@ -106,12 +106,6 @@ LsaIFree_LSAPR_PRIVILEGE_SET(IN PLSAPR_PRIVILEGE_SET Ptr); typedef wchar_t *PSAMPR_SERVER_NAME; typedef void *SAMPR_HANDLE; -typedef struct _SAMPR_ULONG_ARRAY -{ - unsigned long Count; - unsigned long *Element; -} SAMPR_ULONG_ARRAY, *PSAMPR_ULONG_ARRAY; - typedef struct _SAMPR_SID_INFORMATION { PRPC_SID SidPointer; diff --git a/dll/win32/lsasrv/lookup.c b/dll/win32/lsasrv/lookup.c index 19d3c56d3d..b0805ebc0f 100644 --- a/dll/win32/lsasrv/lookup.c +++ b/dll/win32/lsasrv/lookup.c @@ -22,13 +22,6 @@ typedef struct _SAMPR_RETURNED_USTRING_ARRAY PRPC_UNICODE_STRING Element; } SAMPR_RETURNED_USTRING_ARRAY, *PSAMPR_RETURNED_USTRING_ARRAY; -typedef struct _SAMPR_ULONG_ARRAY -{ - unsigned long Count; - unsigned long *Element; -} SAMPR_ULONG_ARRAY, *PSAMPR_ULONG_ARRAY; - - VOID NTAPI SamIFree_SAMPR_RETURNED_USTRING_ARRAY(PSAMPR_RETURNED_USTRING_ARRAY Ptr); diff --git a/dll/win32/lsasrv/lsasrv.h b/dll/win32/lsasrv/lsasrv.h index 47c00d389a..4bc0673c71 100644 --- a/dll/win32/lsasrv/lsasrv.h +++ b/dll/win32/lsasrv/lsasrv.h @@ -79,6 +79,12 @@ typedef struct _LSAP_LOGON_CONTEXT HANDLE ConnectionHandle; } LSAP_LOGON_CONTEXT, *PLSAP_LOGON_CONTEXT; +typedef struct _SAMPR_ULONG_ARRAY +{ + unsigned long Count; + unsigned long *Element; +} SAMPR_ULONG_ARRAY, *PSAMPR_ULONG_ARRAY; + extern SID_IDENTIFIER_AUTHORITY NullSidAuthority; extern SID_IDENTIFIER_AUTHORITY WorldSidAuthority; extern SID_IDENTIFIER_AUTHORITY LocalSidAuthority;

7 years

[reactos] 01/01: [MSAFD] Move the default debug channel to msafd.h.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f6108f3187789fba6fb5c… commit f6108f3187789fba6fb5cd05ce66eee99ee93812 Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 13:24:35 2017 +0100 [MSAFD] Move the default debug channel to msafd.h. --- dll/win32/msafd/misc/dllmain.c | 3 --- dll/win32/msafd/misc/event.c | 3 --- dll/win32/msafd/misc/helpers.c | 3 --- dll/win32/msafd/misc/sndrcv.c | 3 --- dll/win32/msafd/misc/stubs.c | 3 --- dll/win32/msafd/msafd.h | 4 ++++ 6 files changed, 4 insertions(+), 15 deletions(-) diff --git a/dll/win32/msafd/misc/dllmain.c b/dll/win32/msafd/misc/dllmain.c index c2ee828b08..e456796147 100644 --- a/dll/win32/msafd/misc/dllmain.c +++ b/dll/win32/msafd/misc/dllmain.c @@ -15,9 +15,6 @@ #include <winuser.h> #include <wchar.h> -#include <wine/debug.h> -WINE_DEFAULT_DEBUG_CHANNEL(msafd); - HANDLE GlobalHeap; WSPUPCALLTABLE Upcalls; DWORD CatalogEntryId; /* CatalogEntryId for upcalls */ diff --git a/dll/win32/msafd/misc/event.c b/dll/win32/msafd/misc/event.c index 27ceb3cae3..467c039cb4 100644 --- a/dll/win32/msafd/misc/event.c +++ b/dll/win32/msafd/misc/event.c @@ -12,9 +12,6 @@ #include <msafd.h> -#include <wine/debug.h> -WINE_DEFAULT_DEBUG_CHANNEL(msafd); - int WSPAPI WSPEventSelect( diff --git a/dll/win32/msafd/misc/helpers.c b/dll/win32/msafd/misc/helpers.c index aab52bab61..69ee86a1e4 100644 --- a/dll/win32/msafd/misc/helpers.c +++ b/dll/win32/msafd/misc/helpers.c @@ -14,9 +14,6 @@ #include <winreg.h> -#include <wine/debug.h> -WINE_DEFAULT_DEBUG_CHANNEL(msafd); - CRITICAL_SECTION HelperDLLDatabaseLock; LIST_ENTRY HelperDLLDatabaseListHead; diff --git a/dll/win32/msafd/misc/sndrcv.c b/dll/win32/msafd/misc/sndrcv.c index 2f5fd3270d..507bd47256 100644 --- a/dll/win32/msafd/misc/sndrcv.c +++ b/dll/win32/msafd/misc/sndrcv.c @@ -12,9 +12,6 @@ #include <msafd.h> -#include <wine/debug.h> -WINE_DEFAULT_DEBUG_CHANNEL(msafd); - INT WSPAPI WSPAsyncSelect(IN SOCKET Handle, diff --git a/dll/win32/msafd/misc/stubs.c b/dll/win32/msafd/misc/stubs.c index 215cb2c4e3..2ee41289cd 100644 --- a/dll/win32/msafd/misc/stubs.c +++ b/dll/win32/msafd/misc/stubs.c @@ -10,9 +10,6 @@ #include <msafd.h> -#include <wine/debug.h> -WINE_DEFAULT_DEBUG_CHANNEL(msafd); - INT WSPAPI WSPCancelBlockingCall( diff --git a/dll/win32/msafd/msafd.h b/dll/win32/msafd/msafd.h index 1b8d3cf463..3765c2fca7 100644 --- a/dll/win32/msafd/msafd.h +++ b/dll/win32/msafd/msafd.h @@ -28,6 +28,10 @@ #include <tdi.h> #include <afd/shared.h> #include <mswsock.h> + +#include <wine/debug.h> +WINE_DEFAULT_DEBUG_CHANNEL(msafd); + #include "include/helpers.h" extern HANDLE GlobalHeap;

7 years

[reactos] 01/01: [HAL] Add some header guards.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0bae06fa5f5f3e198adcb… commit 0bae06fa5f5f3e198adcb50a86e9ede3649ef22a Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 13:20:22 2017 +0100 [HAL] Add some header guards. --- hal/halx86/apic/apic.h | 2 ++ hal/halx86/apic/tsc.h | 4 ++++ 2 files changed, 6 insertions(+) diff --git a/hal/halx86/apic/apic.h b/hal/halx86/apic/apic.h index 3c5f4234d7..5de22834be 100644 --- a/hal/halx86/apic/apic.h +++ b/hal/halx86/apic/apic.h @@ -1,4 +1,6 @@ +#pragma once + #ifdef _M_AMD64 #define IOAPIC_BASE 0xFFFFFFFFFFFE1000ULL // checkme #define ZERO_VECTOR 0x00 // IRQL 00 diff --git a/hal/halx86/apic/tsc.h b/hal/halx86/apic/tsc.h index 3d06ae12ec..00595a7a69 100644 --- a/hal/halx86/apic/tsc.h +++ b/hal/halx86/apic/tsc.h @@ -1,4 +1,6 @@ +#ifndef _TSC_H_ +#define _TSC_H_ #define NUM_SAMPLES 4 #define MSR_RDTSC 0x10 @@ -16,3 +18,5 @@ VOID NTAPI HalpInitializeTsc(void); #endif #endif + +#endif /* !_TSC_H_ */

7 years

[reactos] 01/01: [VGA] Correct VgaInterpretCmdStream() declaration.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a308735be60121be4ab67… commit a308735be60121be4ab67f0bef53b48e2b6f69d6 Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 13:11:45 2017 +0100 [VGA] Correct VgaInterpretCmdStream() declaration. --- win32ss/drivers/miniport/vga_new/vga.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/win32ss/drivers/miniport/vga_new/vga.c b/win32ss/drivers/miniport/vga_new/vga.c index 22c3c92caa..1b07397419 100644 --- a/win32ss/drivers/miniport/vga_new/vga.c +++ b/win32ss/drivers/miniport/vga_new/vga.c @@ -91,10 +91,10 @@ VgaIsPresent( PHW_DEVICE_EXTENSION HwDeviceExtension ); -VOID +VP_STATUS NTAPI VgaInterpretCmdStream( - PVOID HwDeviceExtension, + PHW_DEVICE_EXTENSION HwDeviceExtension, PUSHORT pusCmdStream );

7 years

[reactos] 01/01: [UUID] Add a PCH.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=69005963fb4b1378f6a66… commit 69005963fb4b1378f6a666c2b8b2483c222740f8 Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 13:04:23 2017 +0100 [UUID] Add a PCH. --- sdk/lib/uuid/CMakeLists.txt | 4 +++- sdk/lib/uuid/precomp.h | 10 ++++++++++ 2 files changed, 13 insertions(+), 1 deletion(-) diff --git a/sdk/lib/uuid/CMakeLists.txt b/sdk/lib/uuid/CMakeLists.txt index d99bc73a35..09efcd927f 100644 --- a/sdk/lib/uuid/CMakeLists.txt +++ b/sdk/lib/uuid/CMakeLists.txt @@ -219,7 +219,9 @@ list(APPEND SOURCE # vcguids.c wdmguids.c otherguids.c - undoc.c) + undoc.c + precomp.h) add_library(uuid ${SOURCE}) +add_pch(uuid precomp.h SOURCE) add_dependencies(uuid psdk) diff --git a/sdk/lib/uuid/precomp.h b/sdk/lib/uuid/precomp.h new file mode 100644 index 0000000000..abe35fac5e --- /dev/null +++ b/sdk/lib/uuid/precomp.h @@ -0,0 +1,10 @@ +#ifndef _UUID_PRECOMP_H_ +#define _UUID_PRECOMP_H_ + +#define WIN32_NO_STATUS +#define WIN32_LEAN_AND_MEAN + +#include <rpc.h> +#include <rpcndr.h> + +#endif /* _UUID_PRECOMP_H_ */

7 years

[reactos] 01/01: [FASTFAT] Fix size checking in VfatGetFileNameInformation()

by Pierre Schweitzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=28343c6c0c5ca7a91778f… commit 28343c6c0c5ca7a91778fcf46f08e1cc6213272c Author: Pierre Schweitzer <pierre(a)reactos.org> AuthorDate: Sat Dec 9 12:45:16 2017 +0100 [FASTFAT] Fix size checking in VfatGetFileNameInformation() --- drivers/filesystems/fastfat/dir.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/filesystems/fastfat/dir.c b/drivers/filesystems/fastfat/dir.c index 284c04bca1..1e657f19fd 100644 --- a/drivers/filesystems/fastfat/dir.c +++ b/drivers/filesystems/fastfat/dir.c @@ -101,17 +101,17 @@ VfatGetFileNameInformation( *Written = 0; Status = STATUS_BUFFER_OVERFLOW; - if (FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName) > BufferLength) + if (FIELD_OFFSET(FILE_NAME_INFORMATION, FileName) > BufferLength) return Status; - if (First || (BufferLength >= FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName) + DirContext->LongNameU.Length)) + if (First || (BufferLength >= FIELD_OFFSET(FILE_NAME_INFORMATION, FileName) + DirContext->LongNameU.Length)) { pInfo->FileNameLength = DirContext->LongNameU.Length; - *Written = FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName); - if (BufferLength > FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName)) + *Written = FIELD_OFFSET(FILE_NAME_INFORMATION, FileName); + if (BufferLength > FIELD_OFFSET(FILE_NAME_INFORMATION, FileName)) { - BytesToCopy = min(DirContext->LongNameU.Length, BufferLength - FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName)); + BytesToCopy = min(DirContext->LongNameU.Length, BufferLength - FIELD_OFFSET(FILE_NAME_INFORMATION, FileName)); RtlCopyMemory(pInfo->FileName, DirContext->LongNameU.Buffer, BytesToCopy);

7 years

[reactos] 01/01: [FASTFAT] Don't mix FileNameInformation and FileNamesInformation (and support the later). This commit fixes weird behavior in our FastFAT implementation. It was mixing two classes: FileNameInformation and FileNamesInformation. It was handling FileNameInformation like FileNamesInformation and was filling buffer with FILE_NAMES_INFORMATION structure instead of FILE_NAME_INFORMATION structure (how many things did that break?!). Also, it wasn't implementing the FileNamesInformation

by Pierre Schweitzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=9f3c80193e5f5d9c04970… commit 9f3c80193e5f5d9c0497033322c515cdb3690e8c Author: Pierre Schweitzer <pierre(a)reactos.org> AuthorDate: Sat Dec 9 12:33:29 2017 +0100 [FASTFAT] Don't mix FileNameInformation and FileNamesInformation (and support the later). This commit fixes weird behavior in our FastFAT implementation. It was mixing two classes: FileNameInformation and FileNamesInformation. It was handling FileNameInformation like FileNamesInformation and was filling buffer with FILE_NAMES_INFORMATION structure instead of FILE_NAME_INFORMATION structure (how many things did that break?!). Also, it wasn't implementing the FileNamesInformation class at all. This is required by ntdll_winetest:directory which doesn't expect it to fail and thus, attempts to read never filled in memory. This commit fixes the winetest crash, and may fix other weird FS behavior. CORE-13367 --- drivers/filesystems/fastfat/dir.c | 51 ++++++++++++++++++++++++++++++++++++++- 1 file changed, 50 insertions(+), 1 deletion(-) diff --git a/drivers/filesystems/fastfat/dir.c b/drivers/filesystems/fastfat/dir.c index db96257281..284c04bca1 100644 --- a/drivers/filesystems/fastfat/dir.c +++ b/drivers/filesystems/fastfat/dir.c @@ -89,6 +89,47 @@ FsdSystemTimeToDosDateTime( static NTSTATUS VfatGetFileNameInformation( + PVFAT_DIRENTRY_CONTEXT DirContext, + PFILE_NAME_INFORMATION pInfo, + ULONG BufferLength, + PULONG Written, + BOOLEAN First) +{ + NTSTATUS Status; + ULONG BytesToCopy = 0; + + *Written = 0; + Status = STATUS_BUFFER_OVERFLOW; + + if (FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName) > BufferLength) + return Status; + + if (First || (BufferLength >= FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName) + DirContext->LongNameU.Length)) + { + pInfo->FileNameLength = DirContext->LongNameU.Length; + + *Written = FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName); + if (BufferLength > FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName)) + { + BytesToCopy = min(DirContext->LongNameU.Length, BufferLength - FIELD_OFFSET(FILE_NAMES_INFORMATION, FileName)); + RtlCopyMemory(pInfo->FileName, + DirContext->LongNameU.Buffer, + BytesToCopy); + *Written += BytesToCopy; + + if (BytesToCopy == DirContext->LongNameU.Length) + { + Status = STATUS_SUCCESS; + } + } + } + + return Status; +} + +static +NTSTATUS +VfatGetFileNamesInformation( PVFAT_DIRENTRY_CONTEXT DirContext, PFILE_NAMES_INFORMATION pInfo, ULONG BufferLength, @@ -622,7 +663,7 @@ DoQuery( { case FileNameInformation: Status = VfatGetFileNameInformation(&DirContext, - (PFILE_NAMES_INFORMATION)Buffer, + (PFILE_NAME_INFORMATION)Buffer, BufferLength, &Written, Buffer0 == NULL); @@ -655,6 +696,14 @@ DoQuery( Buffer0 == NULL); break; + case FileNamesInformation: + Status = VfatGetFileNamesInformation(&DirContext, + (PFILE_NAMES_INFORMATION)Buffer, + BufferLength, + &Written, + Buffer0 == NULL); + break; + default: Status = STATUS_INVALID_INFO_CLASS; break;

7 years

[reactos] 01/01: [SAMLIB] Use precomp.h in dllmain.c.

by Amine Khaldi

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=d086a431cb9a31cb9ed7a… commit d086a431cb9a31cb9ed7a4f592b8c82efbd1b552 Author: Amine Khaldi <amine.khaldi(a)reactos.org> AuthorDate: Sat Dec 9 12:32:18 2017 +0100 [SAMLIB] Use precomp.h in dllmain.c. --- dll/win32/samlib/dllmain.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dll/win32/samlib/dllmain.c b/dll/win32/samlib/dllmain.c index 630d1f998b..99b7275e55 100644 --- a/dll/win32/samlib/dllmain.c +++ b/dll/win32/samlib/dllmain.c @@ -24,7 +24,7 @@ * PROGRAMER: Eric Kohl */ -#include <windef.h> +#include "precomp.h" /* FUNCTIONS *****************************************************************/

7 years

[reactos] 01/01: [SHELL32_APITEST] Add test for SHCreateFileExtractIconW CORE-14082

by Mark Jansen

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4ac511405f69535076bbd… commit 4ac511405f69535076bbdabfc2710891978e5d02 Author: Mark Jansen <mark.jansen(a)reactos.org> AuthorDate: Sat Dec 9 00:51:56 2017 +0100 [SHELL32_APITEST] Add test for SHCreateFileExtractIconW CORE-14082 --- modules/rostests/apitests/shell32/1.ico | Bin 0 -> 4286 bytes modules/rostests/apitests/shell32/CMakeLists.txt | 2 + .../apitests/shell32/SHCreateFileExtractIconW.cpp | 204 +++++++++++++++++++++ modules/rostests/apitests/shell32/resource.rc | 5 + modules/rostests/apitests/shell32/testlist.c | 2 + 5 files changed, 213 insertions(+) diff --git a/modules/rostests/apitests/shell32/1.ico b/modules/rostests/apitests/shell32/1.ico new file mode 100644 index 0000000000..0b2dee3cc6 Binary files /dev/null and b/modules/rostests/apitests/shell32/1.ico differ diff --git a/modules/rostests/apitests/shell32/CMakeLists.txt b/modules/rostests/apitests/shell32/CMakeLists.txt index 65d0f6c504..48c41820c9 100644 --- a/modules/rostests/apitests/shell32/CMakeLists.txt +++ b/modules/rostests/apitests/shell32/CMakeLists.txt @@ -14,10 +14,12 @@ add_executable(shell32_apitest CShellLink.cpp menu.cpp PathResolve.cpp + SHCreateFileExtractIconW.cpp ShellExecuteEx.cpp shelltest.cpp SHParseDisplayName.cpp testlist.c + resource.rc ${CMAKE_CURRENT_BINARY_DIR}/shell32_apitest.def) target_link_libraries(shell32_apitest wine uuid ${PSEH_LIB}) set_module_type(shell32_apitest win32cui) diff --git a/modules/rostests/apitests/shell32/SHCreateFileExtractIconW.cpp b/modules/rostests/apitests/shell32/SHCreateFileExtractIconW.cpp new file mode 100644 index 0000000000..38247dabe3 --- /dev/null +++ b/modules/rostests/apitests/shell32/SHCreateFileExtractIconW.cpp @@ -0,0 +1,204 @@ +/* + * PROJECT: ReactOS API tests + * LICENSE: GPL-2.0+ (https://spdx.org/licenses/GPL-2.0+) + * PURPOSE: Test for SHCreateFileExtractIconW + * COPYRIGHT: Copyright 2017 Mark Jansen (mark.jansen(a)reactos.org) + */ + +#include "shelltest.h" +#include <atlbase.h> +#include <atlcom.h> +ULONG DbgPrint(PCH Format,...); +#include <shellutils.h> +#include <wincon.h> +#include <wingdi.h> + + +HRESULT (STDAPICALLTYPE *pSHCreateFileExtractIconW)(LPCWSTR pszFile, DWORD dwFileAttributes, REFIID riid, void **ppv); + +struct test_data +{ + const WCHAR* Name; + DWORD dwFlags; +}; + +static test_data Tests[] = +{ + { L"xxx.zip", FILE_ATTRIBUTE_NORMAL }, + { L"xxx.zip", FILE_ATTRIBUTE_DIRECTORY }, + { L"xxx.exe", FILE_ATTRIBUTE_NORMAL }, + { L"xxx.exe", FILE_ATTRIBUTE_DIRECTORY }, + { L"xxx.dll", FILE_ATTRIBUTE_NORMAL }, + { L"xxx.dll", FILE_ATTRIBUTE_DIRECTORY }, + { L"xxx.txt", FILE_ATTRIBUTE_NORMAL }, + { L"xxx.txt", FILE_ATTRIBUTE_DIRECTORY }, + { NULL, FILE_ATTRIBUTE_NORMAL }, + { NULL, FILE_ATTRIBUTE_DIRECTORY }, +}; + + +static void ExtractOneBitmap(HBITMAP hbm, CComHeapPtr<BYTE>& data, DWORD& size) +{ + HDC hdc = CreateCompatibleDC(NULL); + HGDIOBJ obj = SelectObject(hdc, hbm); + + CComHeapPtr<BITMAPINFO> pInfoBM; + + pInfoBM.AllocateBytes(sizeof(BITMAPINFOHEADER) + 256 * sizeof(RGBQUAD)); + memset(pInfoBM, 0, sizeof(BITMAPINFOHEADER) + 256 * sizeof(RGBQUAD)); + pInfoBM->bmiHeader.biSize = sizeof(pInfoBM->bmiHeader); + if (!GetDIBits(hdc, hbm, 0, 0, NULL, pInfoBM, DIB_RGB_COLORS)) + return; + + size = pInfoBM->bmiHeader.biSizeImage; + data.Allocate(size); + GetDIBits(hdc, hbm, 0, pInfoBM->bmiHeader.biHeight, data, pInfoBM, DIB_RGB_COLORS); + + SelectObject(hdc, obj); + DeleteDC(hdc); +} + +static bool GetIconData(HICON icon, CComHeapPtr<BYTE>& colorData, DWORD& colorSize, CComHeapPtr<BYTE>& maskData, DWORD& maskSize) +{ + ICONINFO iconinfo; + + if (!GetIconInfo(icon, &iconinfo)) + return false; + + ExtractOneBitmap(iconinfo.hbmColor, colorData, colorSize); + ExtractOneBitmap(iconinfo.hbmMask, maskData, maskSize); + + DeleteObject(iconinfo.hbmColor); + DeleteObject(iconinfo.hbmMask); + + return true; +} + + +START_TEST(SHCreateFileExtractIconW) +{ + WCHAR CurrentModule[MAX_PATH]; + HMODULE shell32 = LoadLibraryA("shell32.dll"); + HICON myIcon; + pSHCreateFileExtractIconW = (HRESULT (__stdcall *)(LPCWSTR, DWORD, REFIID, void **))GetProcAddress(shell32, "SHCreateFileExtractIconW"); + + CoInitialize(NULL); + + GetModuleFileNameW(NULL, CurrentModule, _countof(CurrentModule)); + { + SHFILEINFOW shfi; + ULONG_PTR firet = SHGetFileInfoW(CurrentModule, 0, &shfi, sizeof(shfi), SHGFI_ICON); + myIcon = shfi.hIcon; + if (!firet) + { + skip("Unable to get my own icon\n"); + return; + } + } + + if (!pSHCreateFileExtractIconW) + { + skip("SHCreateFileExtractIconW not available\n"); + return; + } + + for (size_t n = 0; n < _countof(Tests); ++n) + { + test_data& cur = Tests[n]; + bool useMyIcon = false; + + if (cur.Name == NULL) + { + cur.Name = CurrentModule; + useMyIcon = true; + } + + CComPtr<IExtractIconW> spExtract; + HRESULT hr = pSHCreateFileExtractIconW(cur.Name, cur.dwFlags, IID_PPV_ARG(IExtractIconW, &spExtract)); + ok(hr == S_OK, "Expected hr to be S_OK, was 0x%lx for %S(%lx)\n", hr, cur.Name, cur.dwFlags); + + if (!SUCCEEDED(hr)) + continue; + + int ilIndex = -1; + UINT wFlags = 0xdeaddead; + WCHAR Buffer[MAX_PATH]; + + hr = spExtract->GetIconLocation(0, Buffer, _countof(Buffer), &ilIndex, &wFlags); + ok(hr == S_OK, "Expected hr to be S_OK, was 0x%lx for %S(%lx)\n", hr, cur.Name, cur.dwFlags); + if (!SUCCEEDED(hr)) + continue; + + ok(wFlags & (GIL_NOTFILENAME|GIL_PERCLASS), "Expected GIL_NOTFILENAME|GIL_PERCLASS to be set for %S(%lx)\n", cur.Name, cur.dwFlags); + ok(!wcscmp(Buffer, L"*"), "Expected '*', was '%S' for %S(%lx)\n", Buffer, cur.Name, cur.dwFlags); + + HICON ico; + hr = spExtract->Extract(Buffer, ilIndex, &ico, NULL, 0); + + /* Visualize the icon extracted for whoever is stepping through this code. */ + HWND console = GetConsoleWindow(); + SendMessage(console, WM_SETICON, ICON_BIG, (LPARAM)ico); + SendMessage(console, WM_SETICON, ICON_SMALL, (LPARAM)ico); + + CComHeapPtr<BYTE> colorData, maskData; + DWORD colorSize = 0, maskSize = 0; + + GetIconData(ico, colorData, colorSize, maskData, maskSize); + + if (!colorSize || !maskSize) + continue; + + SHFILEINFOW shfi; + ULONG_PTR firet = SHGetFileInfoW(cur.Name, cur.dwFlags, &shfi, sizeof(shfi), SHGFI_USEFILEATTRIBUTES | SHGFI_ICON); + + if (!firet) + continue; + + CComHeapPtr<BYTE> colorDataRef, maskDataRef; + DWORD colorSizeRef = 0, maskSizeRef = 0; + GetIconData(shfi.hIcon, colorDataRef, colorSizeRef, maskDataRef, maskSizeRef); + + ok(colorSizeRef == colorSize, "Expected %lu, was %lu for %S(%lx)\n", colorSizeRef, colorSize, cur.Name, cur.dwFlags); + ok(maskSizeRef == maskSize, "Expected %lu, was %lu for %S(%lx)\n", maskSizeRef, maskSize, cur.Name, cur.dwFlags); + + if (colorSizeRef == colorSize) + { + ok(!memcmp(colorData, colorDataRef, colorSize), "Expected equal colorData for %S(%lx)\n", cur.Name, cur.dwFlags); + } + + if (maskSizeRef == maskSize) + { + ok(!memcmp(maskData, maskDataRef, maskSize), "Expected equal maskData for %S(%lx)\n", cur.Name, cur.dwFlags); + } + + if (useMyIcon) + { + colorDataRef.Free(); + maskDataRef.Free(); + colorSizeRef = maskSizeRef = 0; + GetIconData(myIcon, colorDataRef, colorSizeRef, maskDataRef, maskSizeRef); + + ok(colorSizeRef == colorSize, "Expected %lu, was %lu for %S(%lx)\n", colorSizeRef, colorSize, cur.Name, cur.dwFlags); + ok(maskSizeRef == maskSize, "Expected %lu, was %lu for %S(%lx)\n", maskSizeRef, maskSize, cur.Name, cur.dwFlags); + + if (colorSizeRef == colorSize) + { + /* Incase requested filetype does not match, the exe icon is not used! */ + if (cur.dwFlags == FILE_ATTRIBUTE_DIRECTORY) + { + ok(memcmp(colorData, colorDataRef, colorSize), "Expected colorData to be changed for %S(%lx)\n", cur.Name, cur.dwFlags); + } + else + { + ok(!memcmp(colorData, colorDataRef, colorSize), "Expected equal colorData for %S(%lx)\n", cur.Name, cur.dwFlags); + } + } + + // Mask is not reliable for some reason + //if (maskSizeRef == maskSize) + //{ + // ok(!memcmp(maskData, maskDataRef, maskSize), "Expected equal maskData for %S(%lx)\n", cur.Name, cur.dwFlags); + //} + } + } +} diff --git a/modules/rostests/apitests/shell32/resource.rc b/modules/rostests/apitests/shell32/resource.rc new file mode 100644 index 0000000000..89a0f853a8 --- /dev/null +++ b/modules/rostests/apitests/shell32/resource.rc @@ -0,0 +1,5 @@ +#include <windef.h> + +LANGUAGE LANG_NEUTRAL, SUBLANG_NEUTRAL + +1 ICON "1.ico" diff --git a/modules/rostests/apitests/shell32/testlist.c b/modules/rostests/apitests/shell32/testlist.c index 769f587800..b0a8638046 100644 --- a/modules/rostests/apitests/shell32/testlist.c +++ b/modules/rostests/apitests/shell32/testlist.c @@ -11,6 +11,7 @@ extern void func_CShellDesktop(void); extern void func_CShellLink(void); extern void func_menu(void); extern void func_PathResolve(void); +extern void func_SHCreateFileExtractIconW(void); extern void func_ShellExecuteEx(void); extern void func_SHParseDisplayName(void); @@ -24,6 +25,7 @@ const struct test winetest_testlist[] = { "CShellLink", func_CShellLink }, { "menu", func_menu }, { "PathResolve", func_PathResolve }, + { "SHCreateFileExtractIconW", func_SHCreateFileExtractIconW }, { "ShellExecuteEx", func_ShellExecuteEx }, { "SHParseDisplayName", func_SHParseDisplayName }, { 0, 0 }

7 years

← 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 December 2017