Ros-diffs October 2021

ros-diffs@reactos.org

26 participants
116 discussions

[reactos] 01/01: [ATL][ATL_APITEST] Add test for CPath +small fixes

by Mark Jansen

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e3402aca65925a898e92a… commit e3402aca65925a898e92a604264600657e41ea21 Author: Mark Jansen <mark.jansen(a)reactos.org> AuthorDate: Wed Oct 13 19:33:23 2021 +0200 Commit: Mark Jansen <mark.jansen(a)reactos.org> CommitDate: Sat Oct 30 22:47:57 2021 +0200 [ATL][ATL_APITEST] Add test for CPath +small fixes --- modules/rostests/apitests/atl/CMakeLists.txt | 4 +- modules/rostests/apitests/atl/CPath.cpp | 59 +++++ modules/rostests/apitests/atl/CPath.inl | 283 +++++++++++++++++++++ modules/rostests/apitests/atl/devenv/ATLTest.sln | 13 + modules/rostests/apitests/atl/devenv/CPath.vcxproj | 189 ++++++++++++++ modules/rostests/apitests/atl/testlist.c | 2 + sdk/lib/atl/atlpath.h | 66 +++-- 7 files changed, 587 insertions(+), 29 deletions(-) diff --git a/modules/rostests/apitests/atl/CMakeLists.txt b/modules/rostests/apitests/atl/CMakeLists.txt index bc3184793de..a414eb2d636 100644 --- a/modules/rostests/apitests/atl/CMakeLists.txt +++ b/modules/rostests/apitests/atl/CMakeLists.txt @@ -13,6 +13,8 @@ list(APPEND SOURCE CComVariant.cpp CHeapPtrList.cpp CImage.cpp + CPath.cpp + CPath.inl CRegKey.cpp CSimpleArray.cpp CSimpleMap.cpp @@ -31,6 +33,6 @@ add_executable(atl_apitest target_link_libraries(atl_apitest wine uuid cpprt atl_classes) set_target_cpp_properties(atl_apitest WITH_EXCEPTIONS) set_module_type(atl_apitest win32cui) -add_importlibs(atl_apitest rpcrt4 ole32 oleaut32 msimg32 gdi32 advapi32 user32 msvcrt kernel32 ntdll) +add_importlibs(atl_apitest shlwapi rpcrt4 ole32 oleaut32 msimg32 gdi32 advapi32 user32 msvcrt kernel32 ntdll) add_pch(atl_apitest precomp.h "${PCH_SKIP_SOURCE}") add_rostests_file(TARGET atl_apitest) diff --git a/modules/rostests/apitests/atl/CPath.cpp b/modules/rostests/apitests/atl/CPath.cpp new file mode 100644 index 00000000000..6f8c9581911 --- /dev/null +++ b/modules/rostests/apitests/atl/CPath.cpp @@ -0,0 +1,59 @@ +/* + * PROJECT: ReactOS api tests + * LICENSE: GPL-2.0+ (https://spdx.org/licenses/GPL-2.0+) + * PURPOSE: Test for CPath + * COPYRIGHT: Copyright 2021 Mark Jansen <mark.jansen(a)reactos.org> + */ + +#include <atlpath.h> +#include "resource.h" + +#ifdef HAVE_APITEST + #include <apitest.h> +#else + #include "atltest.h" +#endif + +#undef ok +#undef _T + +#define TEST_NAMEX(name) void test_##name##W() +#define CPathX CPathW +#define CStringX CStringW +#define _X(x) L ## x +#define XCHAR WCHAR +#define dbgstrx(x) wine_dbgstr_w(x) +#define ok ok_("CPathW:\n" __FILE__, __LINE__) +#define GetModuleFileNameX GetModuleFileNameW +#include "CPath.inl" + +#undef CPathX +#undef CStringX +#undef TEST_NAMEX +#undef _X +#undef XCHAR +#undef dbgstrx +#undef ok +#undef GetModuleFileNameX + +#define TEST_NAMEX(name) void test_##name##A() +#define CPathX CPathA +#define CStringX CStringA +#define _X(x) x +#define XCHAR CHAR +#define dbgstrx(x) (const char*)x +#define ok ok_("CPathA:\n" __FILE__, __LINE__) +#define GetModuleFileNameX GetModuleFileNameA +#include "CPath.inl" + +START_TEST(CPath) +{ + test_initW(); + test_initA(); + + test_modifyW(); + test_modifyA(); + + test_is_somethingW(); + test_is_somethingA(); +} diff --git a/modules/rostests/apitests/atl/CPath.inl b/modules/rostests/apitests/atl/CPath.inl new file mode 100644 index 00000000000..e8b919bf7ee --- /dev/null +++ b/modules/rostests/apitests/atl/CPath.inl @@ -0,0 +1,283 @@ + +TEST_NAMEX(init) +{ + CPathX test1; + CPathX test2(_X("C:\\SomePath\\.\\That\\..\\is.txt.exe.bin")); + CPathX test3(test2); + + ok(CStringX("").Compare(test1) == 0, "Expected test1 to be '', was: '%s'\n", dbgstrx(test1)); + ok(CStringX("C:\\SomePath\\.\\That\\..\\is.txt.exe.bin").Compare(test2) == 0, "Expected test2 to be 'C:\\SomePath\\.\\That\\..\\is.txt.exe.bin', was: '%s'\n", dbgstrx(test2)); + ok(CStringX("C:\\SomePath\\.\\That\\..\\is.txt.exe.bin").Compare(test3) == 0, "Expected test3 to be 'C:\\SomePath\\.\\That\\..\\is.txt.exe.bin', was: '%s'\n", dbgstrx(test3)); + + test1 = _X("test"); + test2 = _X("test"); + ok(CStringX("test").Compare(test1) == 0, "Expected test1 to be 'test', was: '%s'\n", dbgstrx(test1)); + ok(CStringX("test").Compare(test2) == 0, "Expected test2 to be 'test', was: '%s'\n", dbgstrx(test1)); + + +#if 0 + // this does not compile: + test3 = test1 + _X("test"); + test3 = test1 + test2; + CPathX test4(test1 + test2); +#endif + // This one compiles, but does not behave as wanted! + CPathX test5(test1 + _X("test")); + ok(CStringX("testtest").Compare(test5) == 0, "Expected test5 to be 'testtest', was: '%s'\n", dbgstrx(test1)); +} + +TEST_NAMEX(modify) +{ + CPathX test(_X("C:\\Some Path\\.\\That\\..\\is.txt.exe.bin")); + CPathX canon; + CPathX empty; + + test.Canonicalize(); + ok(CStringX("C:\\Some Path\\is.txt.exe.bin").Compare(test) == 0, + "Expected test to be 'C:\\Some Path\\is.txt.exe.bin', was: '%s'\n", dbgstrx(test)); + + canon.Canonicalize(); + ok(CStringX("\\").Compare(canon) == 0, + "Expected canon to be '\\', was: '%s'\n", dbgstrx(canon)); + + ok(test.FileExists() == FALSE, "FileExists succeeded for '%s'\n", dbgstrx(test)); + + int ext = test.FindExtension(); + ok(ext == 23, "Expected ext to be 23, was: %i\n", ext); + ext = empty.FindExtension(); + ok(ext == -1, "Expected ext to be -1, was: %i\n", ext); + + int name = test.FindFileName(); + ok(name == 13, "Expected name to be 13, was: %i\n", name); + name = empty.FindFileName(); + ok(name == -1, "Expected name to be -1, was: %i\n", name); + + int drive = test.GetDriveNumber(); + ok(drive == 2, "Expected drive to be 2, was: %i\n", drive); + drive = empty.GetDriveNumber(); + ok(drive == -1, "Expected drive to be -1, was: %i\n", drive); + + int skiproot = test.SkipRoot(); + ok(skiproot == 3, "Expected skiproot to be 3, was: %i\n", skiproot); + +#if 0 + // Does not handle an empty string correctly + skiproot = empty.SkipRoot(); + ok(skiproot == -1, "Expected skiproot to be -1, was: %i\n", skiproot); +#endif + + + test.RemoveExtension(); + ok(CStringX("C:\\Some Path\\is.txt.exe").Compare(test) == 0, + "Expected test to be 'C:\\Some Path\\is.txt.exe', was: '%s'\n", dbgstrx(test)); + + empty.RemoveExtension(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + CStringX extName = test.GetExtension(); + ok(CStringX(".exe").Compare(extName) == 0, + "Expected test.GetExtension() to be '.exe', was: '%s'\n", dbgstrx(extName)); + + extName = empty.GetExtension(); + ok(CStringX("").Compare(extName) == 0, + "Expected empty.GetExtension() to be '', was: '%s'\n", dbgstrx(extName)); + + test.QuoteSpaces(); + ok(CStringX("\"C:\\Some Path\\is.txt.exe\"").Compare(test) == 0, + "Expected test to be '\"C:\\Some Path\\is.txt.exe\"', was: '%s'\n", dbgstrx(test)); + + empty.QuoteSpaces(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + ok(test.RemoveFileSpec(), "RemoveFileSpec failed\n"); + ok(CStringX("\"C:\\Some Path").Compare(test) == 0, + "Expected test to be '\"C:\\Some Path', was: '%s'\n", dbgstrx(test)); + + ok(empty.RemoveFileSpec() == FALSE, "RemoveFileSpec succeeded\n"); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + test.UnquoteSpaces(); + ok(CStringX("\"C:\\Some Path").Compare(test) == 0, + "Expected test to be '\"C:\\Some Path\\is.txt.exe\"', was: '%s'\n", dbgstrx(test)); + + empty.UnquoteSpaces(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + test.m_strPath += _X('"'); + + test.UnquoteSpaces(); + ok(CStringX("C:\\Some Path").Compare(test) == 0, + "Expected test to be 'C:\\Some Path', was: '%s'\n", dbgstrx(test)); + + empty.UnquoteSpaces(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + ok(test.AddExtension(_X(".dummy")), "AddExtension failed\n"); + ok(CStringX("C:\\Some Path.dummy").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.dummy', was: '%s'\n", dbgstrx(test)); + + ok(empty.AddExtension(_X(".dummy")), "AddExtension failed\n"); + ok(CStringX(".dummy").Compare(empty) == 0, + "Expected empty to be '.dummy', was: '%s'\n", dbgstrx(empty)); + empty = _X(""); + + test.AddBackslash(); + ok(CStringX("C:\\Some Path.dummy\\").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.dummy\\', was: '%s'\n", dbgstrx(test)); + + empty.AddBackslash(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + test.AddBackslash(); + ok(CStringX("C:\\Some Path.dummy\\").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.dummy\\', was: '%s'\n", dbgstrx(test)); + + empty.AddBackslash(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + test.RemoveBlanks(); + ok(CStringX("C:\\Some Path.dummy\\").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.dummy\\', was: '%s'\n", dbgstrx(test)); + + empty.RemoveBlanks(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + test = _X(" C:\\Some Path.dummy\\ "); + empty = _X(" "); + + test.RemoveBlanks(); + ok(CStringX("C:\\Some Path.dummy\\").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.dummy\\', was: '%s'\n", dbgstrx(test)); + + empty.RemoveBlanks(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + test.RemoveBackslash(); + ok(CStringX("C:\\Some Path.dummy").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.dummy', was: '%s'\n", dbgstrx(test)); + + empty.RemoveBackslash(); + ok(CStringX("").Compare(empty) == 0, + "Expected empty to be '', was: '%s'\n", dbgstrx(empty)); + + ok(test.RenameExtension(_X(".txt")), "RenameExtension failed\n"); + ok(CStringX("C:\\Some Path.txt").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.txt', was: '%s'\n", dbgstrx(test)); + + ok(empty.RenameExtension(_X(".txt")), "RenameExtension failed\n"); + ok(CStringX(".txt").Compare(empty) == 0, + "Expected empty to be '.txt', was: '%s'\n", dbgstrx(empty)); + + empty = _X(""); + + test += _X("something"); + ok(CStringX("C:\\Some Path.txt\\something").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.txt', was: '%s'\n", dbgstrx(test)); + + empty += _X("something"); + ok(CStringX("something").Compare(empty) == 0, + "Expected empty to be 'something', was: '%s'\n", dbgstrx(empty)); + + ok(test.Append(_X("stuff.txt")), "Append failed\n"); + ok(CStringX("C:\\Some Path.txt\\something\\stuff.txt").Compare(test) == 0, + "Expected test to be 'C:\\Some Path.txt\\something\\stuff.txt', was: '%s'\n", dbgstrx(test)); + + ok(empty.Append(_X("")), "Append failed\n"); + ok(CStringX("something").Compare(empty) == 0, + "Expected empty to be 'something', was: '%s'\n", dbgstrx(empty)); + + ok(test.MatchSpec(_X("stuff.txt")) == FALSE, "MatchSpec succeeded for 'stuff.txt'\n"); + ok(test.MatchSpec(_X("stuff.exe")) == FALSE, "MatchSpec succeeded for 'stuff.exe'\n"); + ok(test.MatchSpec(_X(".txt")) == FALSE, "MatchSpec succeeded for '.txt'\n"); + ok(test.MatchSpec(_X("txt")) == FALSE, "MatchSpec succeeded for 'txt'\n"); + ok(test.MatchSpec(_X("*.txt")), "MatchSpec failed for '*.txt'\n"); + ok(test.MatchSpec(_X("C:\\*.txt")), "MatchSpec failed for 'C:\\*.txt'\n"); + ok(test.MatchSpec(_X("D:\\*.txt")) == FALSE, "MatchSpec succeeded for 'D:\\*.txt'\n"); + + CStringX same("C:\\Some Path.txt\\something\\stuff.txt"); + + const CStringX& constref = test; + ok(same.Compare(constref) == 0, + "Expected constref to be '%s', was: '%s'\n", dbgstrx(same), dbgstrx(constref)); + + + CStringX& ref = test; + ok(same.Compare(ref) == 0, + "Expected ref to be '%s', was: '%s'\n", dbgstrx(same), dbgstrx(ref)); + + const XCHAR* rawptr = test; + ok(same.Compare(rawptr) == 0, + "Expected rawptr to be '%s', was: '%s'\n", dbgstrx(same), dbgstrx(rawptr)); + + test.BuildRoot(5); + ok(CStringX("F:\\").Compare(test) == 0, + "Expected test to be 'F:\\', was: '%s'\n", dbgstrx(test)); + +#if 0 + // Asserts 'iDrive <= 25' + test.BuildRoot(105); + ok(CStringX("f:").Compare(test) == 0, + "Expected test to be 'f:', was: '%s'\n", dbgstrx(test)); +#endif + + test.Combine(_X("C:"), _X("test")); + ok(CStringX("C:\\test").Compare(test) == 0, + "Expected test to be 'C:\\test', was: '%s'\n", dbgstrx(test)); + + test.Combine(_X("C:\\"), _X("\\test\\")); + ok(CStringX("C:\\test\\").Compare(test) == 0, + "Expected test to be 'C:\\test\\', was: '%s'\n", dbgstrx(test)); + + test.Combine(_X("C:\\"), _X("\\test\\\\")); + ok(CStringX("C:\\test\\\\").Compare(test) == 0, + "Expected test to be 'C:\\test\\\\', was: '%s'\n", dbgstrx(test)); +} + +TEST_NAMEX(is_something) +{ + XCHAR Buffer[MAX_PATH] = { 0 }; + GetModuleFileNameX(NULL, Buffer, MAX_PATH); + + CPathX test(Buffer); + + ok(test.IsDirectory() == FALSE, "IsDirectory succeeded for '%s'\n", dbgstrx(test)); + ok(test.IsFileSpec() == FALSE, "IsFileSpec succeeded for '%s'\n", dbgstrx(test)); + ok(test.IsRelative() == FALSE, "IsRelative succeeded for '%s'\n", dbgstrx(test)); + ok(test.IsRoot() == FALSE, "IsRoot succeeded for '%s'\n", dbgstrx(test)); + ok(test.FileExists(), "FileExists failed for '%s'\n", dbgstrx(test)); + + test.StripPath(); + ok(test.IsFileSpec(), "IsFileSpec failed for '%s'\n", dbgstrx(test)); + ok(test.IsRelative(), "IsRelative failed for '%s'\n", dbgstrx(test)); + ok(test.IsRoot() == FALSE, "IsRoot succeeded for '%s'\n", dbgstrx(test)); + + + ok(test.StripToRoot() == FALSE, "StripToRoot succeeded for '%s'\n", dbgstrx(test)); + ok(CStringX("").Compare(test) == 0, + "Expected test to be '', was: '%s'\n", dbgstrx(test)); + + test = Buffer; + + ok(test.StripToRoot(), "StripToRoot failed for '%s'\n", dbgstrx(test)); + ok(test.IsRoot(), "IsRoot failed for '%s'\n", dbgstrx(test)); + + { + CStringX help = test; + ok(help.GetLength() == 3, "IsRoot weird result for '%s'\n", dbgstrx(help)); + } + + test = Buffer; + + test.RemoveFileSpec(); + ok(test.IsDirectory(), "IsDirectory failed for '%s'\n", dbgstrx(test)); +} + diff --git a/modules/rostests/apitests/atl/devenv/ATLTest.sln b/modules/rostests/apitests/atl/devenv/ATLTest.sln index 9811121c7fd..2a923a69b57 100644 --- a/modules/rostests/apitests/atl/devenv/ATLTest.sln +++ b/modules/rostests/apitests/atl/devenv/ATLTest.sln @@ -27,6 +27,8 @@ Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "CComHeapPtr", "CComHeapPtr. EndProject Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "SubclassWindow", "SubclassWindow.vcxproj", "{ABACDAE7-3936-17E1-7525-96C4A7DA4CD2}" EndProject +Project("{8BC9CEB8-8B4A-11D0-8D11-00A0C91BC942}") = "CPath", "CPath.vcxproj", "{7119D446-43C1-425A-91D0-CDFA49B8B59A}" +EndProject Global GlobalSection(SolutionConfigurationPlatforms) = preSolution Debug|x64 = Debug|x64 @@ -131,8 +133,19 @@ Global {ABACDAE7-3936-17E1-7525-96C4A7DA4CD2}.Release|x64.Build.0 = Release|x64 {ABACDAE7-3936-17E1-7525-96C4A7DA4CD2}.Release|x86.ActiveCfg = Release|Win32 {ABACDAE7-3936-17E1-7525-96C4A7DA4CD2}.Release|x86.Build.0 = Release|Win32 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Debug|x64.ActiveCfg = Debug|x64 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Debug|x64.Build.0 = Debug|x64 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Debug|x86.ActiveCfg = Debug|Win32 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Debug|x86.Build.0 = Debug|Win32 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Release|x64.ActiveCfg = Release|x64 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Release|x64.Build.0 = Release|x64 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Release|x86.ActiveCfg = Release|Win32 + {7119D446-43C1-425A-91D0-CDFA49B8B59A}.Release|x86.Build.0 = Release|Win32 EndGlobalSection GlobalSection(SolutionProperties) = preSolution HideSolutionNode = FALSE EndGlobalSection + GlobalSection(ExtensibilityGlobals) = postSolution + SolutionGuid = {6603F212-2099-4D49-8667-FA9DC49C048C} + EndGlobalSection EndGlobal diff --git a/modules/rostests/apitests/atl/devenv/CPath.vcxproj b/modules/rostests/apitests/atl/devenv/CPath.vcxproj new file mode 100644 index 00000000000..b1a43ae24ce --- /dev/null +++ b/modules/rostests/apitests/atl/devenv/CPath.vcxproj @@ -0,0 +1,189 @@ +<?xml version="1.0" encoding="utf-8"?> +<Project DefaultTargets="Build" ToolsVersion="14.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003"> + <ItemGroup Label="ProjectConfigurations"> + <ProjectConfiguration Include="Debug|Win32"> + <Configuration>Debug</Configuration> + <Platform>Win32</Platform> + </ProjectConfiguration> + <ProjectConfiguration Include="Release|Win32"> + <Configuration>Release</Configuration> + <Platform>Win32</Platform> + </ProjectConfiguration> + <ProjectConfiguration Include="Debug|x64"> + <Configuration>Debug</Configuration> + <Platform>x64</Platform> + </ProjectConfiguration> + <ProjectConfiguration Include="Release|x64"> + <Configuration>Release</Configuration> + <Platform>x64</Platform> + </ProjectConfiguration> + </ItemGroup> + <PropertyGroup Label="Globals"> + <ProjectGuid>{7119D446-43C1-425A-91D0-CDFA49B8B59A}</ProjectGuid> + <WindowsTargetPlatformVersion>8.1</WindowsTargetPlatformVersion> + <Keyword>AtlProj</Keyword> + </PropertyGroup> + <Import Project="$(VCTargetsPath)\Microsoft.Cpp.Default.props" /> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'" Label="Configuration"> + <ConfigurationType>Application</ConfigurationType> + <UseDebugLibraries>true</UseDebugLibraries> + <PlatformToolset>v140_xp</PlatformToolset> + <CharacterSet>Unicode</CharacterSet> + </PropertyGroup> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'" Label="Configuration"> + <ConfigurationType>Application</ConfigurationType> + <UseDebugLibraries>false</UseDebugLibraries> + <PlatformToolset>v140_xp</PlatformToolset> + <CharacterSet>Unicode</CharacterSet> + </PropertyGroup> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'" Label="Configuration"> + <ConfigurationType>Application</ConfigurationType> + <UseDebugLibraries>true</UseDebugLibraries> + <PlatformToolset>v140_xp</PlatformToolset> + <CharacterSet>Unicode</CharacterSet> + </PropertyGroup> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'" Label="Configuration"> + <ConfigurationType>Application</ConfigurationType> + <UseDebugLibraries>false</UseDebugLibraries> + <PlatformToolset>v140_xp</PlatformToolset> + <CharacterSet>Unicode</CharacterSet> + </PropertyGroup> + <Import Project="$(VCTargetsPath)\Microsoft.Cpp.props" /> + <ImportGroup Label="ExtensionSettings"> + </ImportGroup> + <ImportGroup Label="Shared"> + </ImportGroup> + <ImportGroup Label="PropertySheets" Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'"> + <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" /> + </ImportGroup> + <ImportGroup Label="PropertySheets" Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> + <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" /> + </ImportGroup> + <ImportGroup Label="PropertySheets" Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> + <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" /> + </ImportGroup> + <ImportGroup Label="PropertySheets" Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> + <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" /> + </ImportGroup> + <PropertyGroup Label="UserMacros" /> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'"> + <IgnoreImportLibrary>true</IgnoreImportLibrary> + <LinkIncremental>true</LinkIncremental> + <IntDir>$(ProjectName)\$(Configuration)\</IntDir> + </PropertyGroup> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> + <IgnoreImportLibrary>true</IgnoreImportLibrary> + <LinkIncremental>true</LinkIncremental> + <IntDir>$(ProjectName)\$(Platform)\$(Configuration)\</IntDir> + </PropertyGroup> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> + <IgnoreImportLibrary>true</IgnoreImportLibrary> + <LinkIncremental>false</LinkIncremental> + <IntDir>$(ProjectName)\$(Configuration)\</IntDir> + </PropertyGroup> + <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> + <IgnoreImportLibrary>true</IgnoreImportLibrary> + <LinkIncremental>false</LinkIncremental> + <IntDir>$(ProjectName)\$(Platform)\$(Configuration)\</IntDir> + </PropertyGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'"> + <ClCompile> + <PrecompiledHeader>NotUsing</PrecompiledHeader> + <WarningLevel>Level3</WarningLevel> + <Optimization>Disabled</Optimization> + <PreprocessorDefinitions>WIN32;_WINDOWS;_DEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + <SDLCheck>true</SDLCheck> + </ClCompile> + <ResourceCompile> + <Culture>0x0409</Culture> + <AdditionalIncludeDirectories>$(IntDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories> + <PreprocessorDefinitions>_DEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + </ResourceCompile> + <Link> + <SubSystem>Console</SubSystem> + <GenerateDebugInformation>true</GenerateDebugInformation> + </Link> + </ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'"> + <ClCompile> + <PrecompiledHeader>NotUsing</PrecompiledHeader> + <WarningLevel>Level3</WarningLevel> + <Optimization>Disabled</Optimization> + <PreprocessorDefinitions>_WINDOWS;_DEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + <SDLCheck>true</SDLCheck> + </ClCompile> + <ResourceCompile> + <Culture>0x0409</Culture> + <AdditionalIncludeDirectories>$(IntDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories> + <PreprocessorDefinitions>_DEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + </ResourceCompile> + <Link> + <SubSystem>Console</SubSystem> + <GenerateDebugInformation>true</GenerateDebugInformation> + </Link> + </ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|Win32'"> + <ClCompile> + <PrecompiledHeader>NotUsing</PrecompiledHeader> + <WarningLevel>Level3</WarningLevel> + <Optimization>MaxSpeed</Optimization> + <PreprocessorDefinitions>WIN32;_WINDOWS;NDEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + <SDLCheck>true</SDLCheck> + </ClCompile> + <ResourceCompile> + <Culture>0x0409</Culture> + <AdditionalIncludeDirectories>$(IntDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories> + <PreprocessorDefinitions>NDEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + </ResourceCompile> + <Link> + <SubSystem>Console</SubSystem> + <GenerateDebugInformation>true</GenerateDebugInformation> + <EnableCOMDATFolding>true</EnableCOMDATFolding> + <OptimizeReferences>true</OptimizeReferences> + </Link> + </ItemDefinitionGroup> + <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Release|x64'"> + <ClCompile> + <PrecompiledHeader>NotUsing</PrecompiledHeader> + <WarningLevel>Level3</WarningLevel> + <Optimization>MaxSpeed</Optimization> + <PreprocessorDefinitions>_WINDOWS;NDEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + <SDLCheck>true</SDLCheck> + </ClCompile> + <ResourceCompile> + <Culture>0x0409</Culture> + <AdditionalIncludeDirectories>$(IntDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories> + <PreprocessorDefinitions>NDEBUG;%(PreprocessorDefinitions)</PreprocessorDefinitions> + </ResourceCompile> + <Link> + <SubSystem>Console</SubSystem> + <GenerateDebugInformation>true</GenerateDebugInformation> + <EnableCOMDATFolding>true</EnableCOMDATFolding> + <OptimizeReferences>true</OptimizeReferences> + </Link> + </ItemDefinitionGroup> + <ItemGroup> + <ClCompile Include="../CPath.cpp"> + <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">MultiThreaded</RuntimeLibrary> + <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Release|x64'">MultiThreaded</RuntimeLibrary> + <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">MultiThreadedDebug</RuntimeLibrary> + <RuntimeLibrary Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">MultiThreadedDebug</RuntimeLibrary> + <PrecompiledHeader Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'">NotUsing</PrecompiledHeader> + <PrecompiledHeader Condition="'$(Configuration)|$(Platform)'=='Release|Win32'">NotUsing</PrecompiledHeader> + <PrecompiledHeader Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">NotUsing</PrecompiledHeader> + <PrecompiledHeader Condition="'$(Configuration)|$(Platform)'=='Release|x64'">NotUsing</PrecompiledHeader> + </ClCompile> + </ItemGroup> + <ItemGroup> + <ClInclude Include="..\resource.h" /> + </ItemGroup> + <ItemGroup> + <ResourceCompile Include="..\atl_apitest.rc" /> + </ItemGroup> + <ItemGroup> + <None Include="..\cpath.inl" /> + </ItemGroup> + <Import Project="$(VCTargetsPath)\Microsoft.Cpp.targets" /> + <ImportGroup Label="ExtensionTargets"> + </ImportGroup> +</Project> \ No newline at end of file diff --git a/modules/rostests/apitests/atl/testlist.c b/modules/rostests/apitests/atl/testlist.c index 213ce2419c4..510b5f4d834 100644 --- a/modules/rostests/apitests/atl/testlist.c +++ b/modules/rostests/apitests/atl/testlist.c @@ -12,6 +12,7 @@ extern void func_CComQIPtr(void); extern void func_CComVariant(void); extern void func_CHeapPtrList(void); extern void func_CImage(void); +extern void func_CPath(void); extern void func_CRegKey(void); extern void func_CSimpleArray(void); extern void func_CSimpleMap(void); @@ -31,6 +32,7 @@ const struct test winetest_testlist[] = { "CComVariant", func_CComVariant }, { "CHeapPtrList", func_CHeapPtrList }, { "CImage", func_CImage }, + { "CPath", func_CPath }, { "CRegKey", func_CRegKey }, { "CSimpleArray", func_CSimpleArray }, { "CSimpleMap", func_CSimpleMap }, diff --git a/sdk/lib/atl/atlpath.h b/sdk/lib/atl/atlpath.h index 942b47f638f..6c5c03df68a 100644 --- a/sdk/lib/atl/atlpath.h +++ b/sdk/lib/atl/atlpath.h @@ -6,8 +6,6 @@ #ifndef __ATLPATH_H__ #define __ATLPATH_H__ -// WARNING: Untested code - #pragma once #include <atlcore.h> @@ -21,36 +19,36 @@ template<typename StringType> class CPathT { // const - inline BOOL PathFileExistsX(LPCSTR pszPath) { return PathFileExistsA(pszPath); } - inline BOOL PathFileExistsX(LPCWSTR pszPath) { return PathFileExistsW(pszPath); } - inline LPCSTR PathFindExtensionX(LPCSTR pszPath) { return PathFindExtensionA(pszPath); } - inline LPCWSTR PathFindExtensionX(LPCWSTR pszPath) { return PathFindExtensionW(pszPath); } - inline LPCSTR PathFindFileNameX(LPCSTR pszPath) { return PathFindFileNameA(pszPath); } - inline LPCWSTR PathFindFileNameX(LPCWSTR pszPath) { return PathFindFileNameW(pszPath); } - inline int PathGetDriveNumberX(LPCSTR pszPath) { return PathGetDriveNumberA(pszPath); } - inline int PathGetDriveNumberX(LPCWSTR pszPath) { return PathGetDriveNumberW(pszPath); } - inline BOOL PathIsDirectoryX(LPCSTR pszPath) { return PathIsDirectoryA(pszPath); } - inline BOOL PathIsDirectoryX(LPCWSTR pszPath) { return PathIsDirectoryW(pszPath); } - inline BOOL PathIsFileSpecX(LPCSTR pszPath) { return PathIsFileSpecA(pszPath); } - inline BOOL PathIsFileSpecX(LPCWSTR pszPath) { return PathIsFileSpecW(pszPath); } + inline BOOL PathFileExistsX(LPCSTR pszPath) const { return PathFileExistsA(pszPath); } + inline BOOL PathFileExistsX(LPCWSTR pszPath) const { return PathFileExistsW(pszPath); } + inline LPCSTR PathFindExtensionX(LPCSTR pszPath) const { return PathFindExtensionA(pszPath); } + inline LPCWSTR PathFindExtensionX(LPCWSTR pszPath) const { return PathFindExtensionW(pszPath); } + inline LPCSTR PathFindFileNameX(LPCSTR pszPath) const { return PathFindFileNameA(pszPath); } + inline LPCWSTR PathFindFileNameX(LPCWSTR pszPath) const { return PathFindFileNameW(pszPath); } + inline int PathGetDriveNumberX(LPCSTR pszPath) const { return PathGetDriveNumberA(pszPath); } + inline int PathGetDriveNumberX(LPCWSTR pszPath) const { return PathGetDriveNumberW(pszPath); } + inline BOOL PathIsDirectoryX(LPCSTR pszPath) const { return PathIsDirectoryA(pszPath); } + inline BOOL PathIsDirectoryX(LPCWSTR pszPath) const { return PathIsDirectoryW(pszPath); } + inline BOOL PathIsFileSpecX(LPCSTR pszPath) const { return PathIsFileSpecA(pszPath); } + inline BOOL PathIsFileSpecX(LPCWSTR pszPath) const { return PathIsFileSpecW(pszPath); } inline BOOL PathIsPrefixX(LPCSTR pszPath, LPCSTR pszPrefix) { return PathIsPrefixA(pszPath, pszPrefix); } inline BOOL PathIsPrefixX(LPCWSTR pszPath, LPCWSTR pszPrefix) { return PathIsPrefixW(pszPath, pszPrefix); } - inline BOOL PathIsRelativeX(LPCSTR pszPath) { return PathIsRelativeA(pszPath); } - inline BOOL PathIsRelativeX(LPCWSTR pszPath) { return PathIsRelativeW(pszPath); } - inline BOOL PathIsRootX(LPCSTR pszPath) { return PathIsRootA(pszPath); } - inline BOOL PathIsRootX(LPCWSTR pszPath) { return PathIsRootW(pszPath); } - inline BOOL PathIsSameRootX(LPCSTR pszPath, LPCSTR pszOther) { return PathIsSameRootA(pszPath, pszOther); } - inline BOOL PathIsSameRootX(LPCWSTR pszPath, LPCWSTR pszOther) { return PathIsSameRootW(pszPath, pszOther); } + inline BOOL PathIsRelativeX(LPCSTR pszPath) const { return PathIsRelativeA(pszPath); } + inline BOOL PathIsRelativeX(LPCWSTR pszPath) const { return PathIsRelativeW(pszPath); } + inline BOOL PathIsRootX(LPCSTR pszPath) const { return PathIsRootA(pszPath); } + inline BOOL PathIsRootX(LPCWSTR pszPath) const { return PathIsRootW(pszPath); } + inline BOOL PathIsSameRootX(LPCSTR pszPath, LPCSTR pszOther) const { return PathIsSameRootA(pszPath, pszOther); } + inline BOOL PathIsSameRootX(LPCWSTR pszPath, LPCWSTR pszOther) const { return PathIsSameRootW(pszPath, pszOther); } inline BOOL PathIsUNCX(LPCSTR pszPath) { return PathIsUNCA(pszPath); } inline BOOL PathIsUNCX(LPCWSTR pszPath) { return PathIsUNCW(pszPath); } inline BOOL PathIsUNCServerX(LPCSTR pszPath) { return PathIsUNCServerA(pszPath); } inline BOOL PathIsUNCServerX(LPCWSTR pszPath) { return PathIsUNCServerW(pszPath); } inline BOOL PathIsUNCServerShareX(LPCSTR pszPath) { return PathIsUNCServerShareA(pszPath); } inline BOOL PathIsUNCServerShareX(LPCWSTR pszPath) { return PathIsUNCServerShareW(pszPath); } - inline BOOL PathMatchSpecX(LPCSTR pszPath, LPCSTR pszSpec) { return PathMatchSpecA(pszPath, pszSpec); } - inline BOOL PathMatchSpecX(LPCWSTR pszPath, LPCWSTR pszSpec) { return PathMatchSpecW(pszPath, pszSpec); } - inline LPCSTR PathSkipRootX(LPCSTR pszPath) { return PathSkipRootA(pszPath); } - inline LPCWSTR PathSkipRootX(LPCWSTR pszPath) { return PathSkipRootW(pszPath); } + inline BOOL PathMatchSpecX(LPCSTR pszPath, LPCSTR pszSpec) const { return PathMatchSpecA(pszPath, pszSpec); } + inline BOOL PathMatchSpecX(LPCWSTR pszPath, LPCWSTR pszSpec) const { return PathMatchSpecW(pszPath, pszSpec); } + inline LPCSTR PathSkipRootX(LPCSTR pszPath) const { return PathSkipRootA(pszPath); } + inline LPCWSTR PathSkipRootX(LPCWSTR pszPath) const { return PathSkipRootW(pszPath); } // non-const inline void PathAddBackslashX(LPSTR pszPath) { PathAddBackslashA(pszPath); } @@ -196,12 +194,22 @@ public: int FindExtension() const { - return PathFindExtensionX(m_strPath); + PCXSTR extension = PathFindExtensionX(m_strPath); + if (*extension == '\0') + return -1; + + PCXSTR str = m_strPath; + return (int)(extension - str); } int FindFileName() const { - return PathFindFileNameX(m_strPath); + PCXSTR filename = PathFindFileNameX(m_strPath); + if (*filename == '\0') + return -1; + + PCXSTR str = m_strPath; + return (int)(filename - str); } int GetDriveNumber() const @@ -333,7 +341,8 @@ public: int SkipRoot() const { - return PathSkipRootX(m_strPath); + PCXSTR str = m_strPath; + return (int)(PathSkipRootX(m_strPath) - str); } void StripPath() @@ -375,7 +384,8 @@ public: CPathT<StringType>& operator+=(PCXSTR pszMore) { - m_strPath += pszMore; + Append(pszMore); + return *this; } };

2 years, 11 months

[reactos] 01/01: [PARTMGR] Mark partitions as removable if device is removable

by Hervé Poussineau

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=06fc9c43ed9363628ee1e… commit 06fc9c43ed9363628ee1eb43985f73690d0ab261 Author: Hervé Poussineau <hpoussin(a)reactos.org> AuthorDate: Sat Oct 30 08:44:26 2021 +0200 Commit: Hervé Poussineau <hpoussin(a)reactos.org> CommitDate: Sat Oct 30 08:47:38 2021 +0200 [PARTMGR] Mark partitions as removable if device is removable CORE-17392 --- drivers/storage/partmgr/partmgr.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/drivers/storage/partmgr/partmgr.c b/drivers/storage/partmgr/partmgr.c index 36c09f26bea..c606a46ead0 100644 --- a/drivers/storage/partmgr/partmgr.c +++ b/drivers/storage/partmgr/partmgr.c @@ -269,6 +269,10 @@ PartMgrUpdatePartitionDevices( continue; } + // mark partition as removable if parent device is removable + if (FdoExtension->LowerDevice->Characteristics & FILE_REMOVABLE_MEDIA) + partitionDevice->Characteristics |= FILE_REMOVABLE_MEDIA; + totalPartitions++; // insert the structure to the partition list

2 years, 11 months

[reactos] 04/04: [WIN32K] Fix uninitialized bResult in IntExtTextOutW.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=3a323c3de0b9721cb1981… commit 3a323c3de0b9721cb1981080ecee68c30388e5ef Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Fri Oct 15 22:35:32 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Fri Oct 29 22:06:49 2021 -0400 [WIN32K] Fix uninitialized bResult in IntExtTextOutW. Powered by clang-cl. --- win32ss/gdi/ntgdi/freetype.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/win32ss/gdi/ntgdi/freetype.c b/win32ss/gdi/ntgdi/freetype.c index 55bb3612e45..b56aab526d6 100644 --- a/win32ss/gdi/ntgdi/freetype.c +++ b/win32ss/gdi/ntgdi/freetype.c @@ -6168,6 +6168,7 @@ IntExtTextOutW( { DPRINT1("Failed to render glyph! [index: %d]\n", glyph_index); IntUnLockFreeType(); + bResult = FALSE; goto Cleanup; } @@ -6243,6 +6244,7 @@ IntExtTextOutW( { DPRINT1("Failed to load and render glyph! [index: %d]\n", glyph_index); IntUnLockFreeType(); + bResult = FALSE; goto Cleanup; } @@ -6256,6 +6258,7 @@ IntExtTextOutW( { DPRINT1("Failed to render glyph! [index: %d]\n", glyph_index); IntUnLockFreeType(); + bResult = FALSE; goto Cleanup; }

2 years, 11 months

[reactos] 03/04: [WIN32K] Mark default as unreachable.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=4bc95812fd70ea382e234… commit 4bc95812fd70ea382e234b3c8e2e043f51fcae3a Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Fri Oct 15 22:31:38 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Fri Oct 29 22:06:49 2021 -0400 [WIN32K] Mark default as unreachable. Powered by clang-cl. --- win32ss/user/ntuser/defwnd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/win32ss/user/ntuser/defwnd.c b/win32ss/user/ntuser/defwnd.c index 4bfe24e3d55..660fd4c1494 100644 --- a/win32ss/user/ntuser/defwnd.c +++ b/win32ss/user/ntuser/defwnd.c @@ -486,8 +486,7 @@ DefWndGetIcon(PWND pWnd, WPARAM wParam, LPARAM lParam) case ICON_SMALL2: hIconRet = UserGetProp(pWnd, gpsi->atomIconSmProp, TRUE); break; - default: - break; + DEFAULT_UNREACHABLE; } return (LRESULT)hIconRet; }

2 years, 11 months

[reactos] 02/04: [WIN32K] Remove unnecessary if in NtUserGetGUIThreadInfo.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=3235e35dee1fb20b462e5… commit 3235e35dee1fb20b462e55a60fcafef63fe70638 Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Fri Oct 15 22:30:34 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Fri Oct 29 22:06:49 2021 -0400 [WIN32K] Remove unnecessary if in NtUserGetGUIThreadInfo. This condition is already checked above. Powered by clang-cl. --- win32ss/user/ntuser/misc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/win32ss/user/ntuser/misc.c b/win32ss/user/ntuser/misc.c index f8c0d3e1ec3..10fa0144f16 100644 --- a/win32ss/user/ntuser/misc.c +++ b/win32ss/user/ntuser/misc.c @@ -426,7 +426,7 @@ NtUserGetGUIThreadInfo( MsgQueue = W32Thread->MessageQueue; else { - if ( Desktop ) MsgQueue = Desktop->ActiveMessageQueue; + MsgQueue = Desktop->ActiveMessageQueue; } } else

2 years, 11 months

[reactos] 01/04: [WIN32K] Fix uninitialized hNewMenu use in MENU_DoNextMenu.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=42657493ef5b1dd7e7718… commit 42657493ef5b1dd7e7718ecf5f3c84606e5d8e9f Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Fri Oct 15 22:30:04 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Fri Oct 29 22:06:46 2021 -0400 [WIN32K] Fix uninitialized hNewMenu use in MENU_DoNextMenu. See also: Wine's implementation. Powered by clang-cl. --- win32ss/user/ntuser/menu.c | 1 + 1 file changed, 1 insertion(+) diff --git a/win32ss/user/ntuser/menu.c b/win32ss/user/ntuser/menu.c index 3c006fcd5a6..fb169b6e472 100644 --- a/win32ss/user/ntuser/menu.c +++ b/win32ss/user/ntuser/menu.c @@ -3813,6 +3813,7 @@ static LRESULT FASTCALL MENU_DoNextMenu(MTRACKER* pmt, UINT Vk, UINT wFlags) /* switch to the system menu */ MenuTmp = get_win_sys_menu(hNewWnd); if (MenuTmp) hNewMenu = UserHMGetHandle(MenuTmp); + else hNewMenu = NULL; } else return FALSE;

2 years, 11 months

[reactos] 01/01: [NETSTAT] Fix coverity #1477187 "Double free" (#4069)

by Efe Itietie

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0e75fc924045d4bcc8a2d… commit 0e75fc924045d4bcc8a2d859a905f251711cb1ce Author: Efe Itietie <omoiti00(a)gmail.com> AuthorDate: Mon Oct 25 11:08:03 2021 -0700 Commit: Stanislav Motylkov <x86corez(a)gmail.com> CommitDate: Thu Oct 28 17:26:09 2021 +0300 [NETSTAT] Fix coverity #1477187 "Double free" (#4069) Within the current ShowTcpTable function logic, tcpTable would be freed twice. The bug was introduced in b695971c. Remove the second tcpTable free and fix coverity #1477187. CORE-17831 --- base/applications/network/netstat/netstat.c | 1 - 1 file changed, 1 deletion(-) diff --git a/base/applications/network/netstat/netstat.c b/base/applications/network/netstat/netstat.c index bee7f834fd0..b1d177effec 100644 --- a/base/applications/network/netstat/netstat.c +++ b/base/applications/network/netstat/netstat.c @@ -434,7 +434,6 @@ BOOL ShowTcpTable(VOID) { ConResPrintf(StdErr, IDS_ERROR_TCP_SNAPSHOT); DoFormatMessage(error); - HeapFree(GetProcessHeap(), 0, tcpTable); return FALSE; }

2 years, 11 months

[reactos] 06/06: [CODEOWNERS] Don't automatically add me to apitests and CMake PRs.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f4ccc518e35fe7747cfdf… commit f4ccc518e35fe7747cfdf0d92173d0b8cac29222 Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Sat Oct 16 11:33:53 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Wed Oct 27 19:27:33 2021 -0400 [CODEOWNERS] Don't automatically add me to apitests and CMake PRs. Still feel free to request my review explicitly if desired. --- CODEOWNERS | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/CODEOWNERS b/CODEOWNERS index 13d3ad435c6..86f8aa11961 100644 --- a/CODEOWNERS +++ b/CODEOWNERS @@ -39,7 +39,7 @@ ## As responsible, he may refuse a local patch if you did not try to ## upstream your changes. -## If you want to get listed in that file, either put yourself in the +## If you want to get listed in this file, either put yourself in the ## file and push it, or open a pull request. You can also ask a person ## who has push access to add yourself. @@ -121,8 +121,8 @@ # R: learn-more, learn_more, Mark Jansen # R: ThFabba, Thomas Faber # S: Maintained -/sdk/cmake/ @learn-more @ThFabba -*.cmake @learn-more @ThFabba +/sdk/cmake/ @learn-more +*.cmake @learn-more # File Patch API # M: learn-more, learn_more, Mark Jansen @@ -225,7 +225,7 @@ # R: learn-more, learn_more, Mark Jansen # R: ThFabba, Thomas Faber # S: Maintained -/modules/rostests/apitests/ @learn-more @ThFabba +/modules/rostests/apitests/ @learn-more # ReactOS Kernel-Mode Tests # M: ThFabba, Thomas Faber

2 years, 11 months

[reactos] 05/06: [USBUHCI] Fix always-false HcFlavor checks.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=fb3e2c7c8b8111bbbf0b6… commit fb3e2c7c8b8111bbbf0b6881573a203d8c8ec659 Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Sat Oct 9 16:01:47 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Wed Oct 27 19:27:33 2021 -0400 [USBUHCI] Fix always-false HcFlavor checks. Powered by clang-cl. --- drivers/usb/usbuhci/roothub.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/usb/usbuhci/roothub.c b/drivers/usb/usbuhci/roothub.c index 286e5c3fd3d..7c166122c68 100644 --- a/drivers/usb/usbuhci/roothub.c +++ b/drivers/usb/usbuhci/roothub.c @@ -195,13 +195,13 @@ UhciRHPortResetComplete(IN PVOID uhciExtension, PortControl.PortEnableDisableChange = 1; WRITE_PORT_USHORT(PortControlRegister, PortControl.AsUSHORT); - if (UhciExtension->HcFlavor == UHCI_VIA && - UhciExtension->HcFlavor == UHCI_VIA_x01 && - UhciExtension->HcFlavor == UHCI_VIA_x02 && - UhciExtension->HcFlavor == UHCI_VIA_x03 && + if (UhciExtension->HcFlavor == UHCI_VIA || + UhciExtension->HcFlavor == UHCI_VIA_x01 || + UhciExtension->HcFlavor == UHCI_VIA_x02 || + UhciExtension->HcFlavor == UHCI_VIA_x03 || UhciExtension->HcFlavor == UHCI_VIA_x04) { - DPRINT("UhciRHPortResetComplete: Via chip. FIXME\n"); + DPRINT1("UhciRHPortResetComplete: Via chip. FIXME\n"); DbgBreakPoint(); return; } @@ -266,10 +266,10 @@ UhciRHSetFeaturePortReset(IN PVOID uhciExtension, UhciExtension->ResetPortMask = ResetPortMask | PortBit; - if (UhciExtension->HcFlavor == UHCI_VIA && - UhciExtension->HcFlavor == UHCI_VIA_x01 && - UhciExtension->HcFlavor == UHCI_VIA_x02 && - UhciExtension->HcFlavor == UHCI_VIA_x03 && + if (UhciExtension->HcFlavor == UHCI_VIA || + UhciExtension->HcFlavor == UHCI_VIA_x01 || + UhciExtension->HcFlavor == UHCI_VIA_x02 || + UhciExtension->HcFlavor == UHCI_VIA_x03 || UhciExtension->HcFlavor == UHCI_VIA_x04) { DPRINT1("UhciRHSetFeaturePortReset: Via chip. FIXME\n");

2 years, 11 months

[reactos] 04/06: [USBEHCI] Avoid uninitialized return status in EHCI_ReopenEndpoint.

by Thomas Faber

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=555eec03061a28858af8e… commit 555eec03061a28858af8e4e0909f063a1f8940ec Author: Thomas Faber <thomas.faber(a)reactos.org> AuthorDate: Sat Oct 9 16:01:06 2021 -0400 Commit: Thomas Faber <thomas.faber(a)reactos.org> CommitDate: Wed Oct 27 19:27:33 2021 -0400 [USBEHCI] Avoid uninitialized return status in EHCI_ReopenEndpoint. Powered by clang-cl. --- drivers/usb/usbehci/usbehci.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/usbehci/usbehci.c b/drivers/usb/usbehci/usbehci.c index bdfd889ff1f..d7ce4aae37a 100644 --- a/drivers/usb/usbehci/usbehci.c +++ b/drivers/usb/usbehci/usbehci.c @@ -603,6 +603,7 @@ EHCI_ReopenEndpoint(IN PVOID ehciExtension, QH->sqh.HwQH.EndpointCaps.HubAddr = EndpointProperties->HubAddr; + MPStatus = MP_STATUS_SUCCESS; break; default:

2 years, 11 months

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs October 2021