- From 957e2274823fe1cc8e60eb2c1639617d6b8d4642 Mon Sep 17 00:00:00 2001
- From: Roland Mainz <roland.mainz@nrubsig.org>
- Date: Thu, 21 Dec 2023 17:23:06 +0100
- Subject: [PATCH 1/5] cygwin/Makefile: Binary tarball should have date
- first+print full tarball path
- The Cygwin binary tarball filename should have the date first,
- so the files can get correctly sorted the default (alphabetic) sort
- orderiung of download areas.
- Also, $ make -f cygwin/Makefile bintarball # should print a message
- with the full path of the tarball, so developers can use copy&paste
- in xterm/konsole.
- Signed-off-by: Cedric Blancher <cedric.blancher@gmail.com>
- ---
- cygwin/Makefile | 10 +++++++---
- 1 file changed, 7 insertions(+), 3 deletions(-)
- diff --git a/cygwin/Makefile b/cygwin/Makefile
- index e9207f6..4f62e85 100644
- --- a/cygwin/Makefile
- +++ b/cygwin/Makefile
- @@ -101,16 +101,20 @@ installdest: $(VS_BUILD_DIR)/nfsd.exe \
- "$(DESTDIR)/cygdrive/c/cygwin64/sbin/"
- bintarball: installdest
- + set -o errexit ; \
- + base_filename="msnfs41client_cygwin_binaries_$$(date +%Y%m%d_%Hh%Mm)_git$$(git rev-parse --short HEAD)" ; \
- ( \
- - base_filename="msnfs41client_cygwin_binaries_git$$(git rev-parse --short HEAD)_$$(date +%Y%m%d_%Hh%Mm)" ; \
- - cd destdir && \
- + cd "$(DESTDIR)" && \
- cp "../cygwin/README.bintarball.txt" "$${base_filename}.readme" && \
- tar -cvf - \
- --owner=SYSTEM:18 \
- --group=SYSTEM:18 \
- cygdrive/c/cygwin64 | \
- bzip2 -9 >"$${base_filename}.tar.bz2" \
- - )
- + ) ; \
- + printf "\n#\n# tarball is ready now\n#\n" ; \
- + ls -l "$(DESTDIR)/$${base_filename}.tar.bz2" ; \
- + ls -l "$(DESTDIR)/$${base_filename}.readme"
- #
- # clean destdir/, but only if nfsd*.exe does not use it right now
- --
- 2.42.1
- From 68a8c270351c8d9170eb9ebaed2a9f99f3a60820 Mon Sep 17 00:00:00 2001
- From: Roland Mainz <roland.mainz@nrubsig.org>
- Date: Thu, 21 Dec 2023 18:45:25 +0100
- Subject: [PATCH 2/5] VC19 build: Switch to ISO C17
- Switch default C language from legacy MSVC to VC19 ISO C17,
- this should give better performance, better compiler errors
- and warnings, and less suprises with C compiler behaviour.
- Signed-off-by: Cedric Blancher <cedric.blancher@gmail.com>
- ---
- build.vc19/libtirpc/libtirpc.vcxproj | 4 ++++
- build.vc19/nfs41_driver/nfs41_driver.vcxproj | 17 +++++++++++++++++
- build.vc19/nfs41_np/nfs41_np.vcxproj | 4 ++++
- build.vc19/nfs_install/nfs_install.vcxproj | 4 ++++
- build.vc19/nfs_mount/nfs_mount.vcxproj | 4 ++++
- build.vc19/nfsd/nfsd.vcxproj | 4 ++++
- daemon/nfs41_daemon.c | 4 ++++
- daemon/nfs41_daemon.h | 4 ++++
- dll/nfs41_np.c | 4 ++++
- install/nfs_install.c | 4 ++++
- libtirpc/libtirpc/libtirpc.rc | 2 +-
- libtirpc/src/gettimeofday.c | 2 +-
- libtirpc/tirpc/wintirpc.h | 5 +++++
- mount/mount.c | 3 +++
- sys/nfs41_driver.c | 8 ++++++++
- 15 files changed, 71 insertions(+), 2 deletions(-)
- diff --git a/build.vc19/libtirpc/libtirpc.vcxproj b/build.vc19/libtirpc/libtirpc.vcxproj
- index 43881fd..c32475a 100644
- --- a/build.vc19/libtirpc/libtirpc.vcxproj
- +++ b/build.vc19/libtirpc/libtirpc.vcxproj
- @@ -89,6 +89,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;PORTMAP;_WINSOCK_DEPRECATED_NO_WARNINGS;WIN32;_DEBUG;_WINDOWS;_USRDLL;LIBTIRPC_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- @@ -105,6 +106,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;PORTMAP;_WINSOCK_DEPRECATED_NO_WARNINGS;_DEBUG;_WINDOWS;_USRDLL;LIBTIRPC_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- @@ -123,6 +125,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;PORTMAP;_WINSOCK_DEPRECATED_NO_WARNINGS;WIN32;NDEBUG;_WINDOWS;_USRDLL;LIBTIRPC_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- @@ -143,6 +146,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;PORTMAP;_WINSOCK_DEPRECATED_NO_WARNINGS;NDEBUG;_WINDOWS;_USRDLL;LIBTIRPC_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- diff --git a/build.vc19/nfs41_driver/nfs41_driver.vcxproj b/build.vc19/nfs41_driver/nfs41_driver.vcxproj
- index 31880bb..33e4053 100644
- --- a/build.vc19/nfs41_driver/nfs41_driver.vcxproj
- +++ b/build.vc19/nfs41_driver/nfs41_driver.vcxproj
- @@ -42,6 +42,7 @@
- <Configuration>Debug</Configuration>
- <Platform Condition="'$(Platform)' == ''">Win32</Platform>
- <RootNamespace>nfs41_driver</RootNamespace>
- + <WindowsTargetPlatformVersion>$(LatestTargetPlatformVersion)</WindowsTargetPlatformVersion>
- </PropertyGroup>
- <Import Project="$(VCTargetsPath)\Microsoft.Cpp.Default.props" />
- <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|Win32'" Label="Configuration">
- @@ -137,6 +138,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -147,6 +150,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -157,6 +162,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -167,6 +174,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -177,6 +186,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -187,6 +198,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -197,6 +210,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- @@ -207,6 +222,8 @@
- <AdditionalIncludeDirectories>..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- <WarningLevel>Level3</WarningLevel>
- <PreprocessorDefinitions>EXPLODE_POOLTAGS;MONOLITHIC_MINIRDR;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- + <AdditionalOptions>/std:c17 /Zc:preprocessor- /kernel /wd5104</AdditionalOptions>
- </ClCompile>
- <Link>
- <AdditionalDependencies>$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ntoskrnl.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\hal.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\bufferoverflowk.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\ksecdd.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rxce.lib;$(WindowsSdkDir)lib\10.0.19041.0\km\$(DDKPlatform)\rdbsslib.lib;$(WindowsSdkDir)lib\win7\km\$(DDKPlatform)\copysup.lib</AdditionalDependencies>
- diff --git a/build.vc19/nfs41_np/nfs41_np.vcxproj b/build.vc19/nfs41_np/nfs41_np.vcxproj
- index a31a57d..60d3000 100644
- --- a/build.vc19/nfs41_np/nfs41_np.vcxproj
- +++ b/build.vc19/nfs41_np/nfs41_np.vcxproj
- @@ -89,6 +89,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>WIN32;_DEBUG;_WINDOWS;_USRDLL;NFS41_NP_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- @@ -104,6 +105,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>_DEBUG;_WINDOWS;_USRDLL;NFS41_NP_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- @@ -121,6 +123,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>WIN32;NDEBUG;_WINDOWS;_USRDLL;NFS41_NP_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- @@ -140,6 +143,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>NDEBUG;_WINDOWS;_USRDLL;NFS41_NP_EXPORTS;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Windows</SubSystem>
- diff --git a/build.vc19/nfs_install/nfs_install.vcxproj b/build.vc19/nfs_install/nfs_install.vcxproj
- index 8cb69bd..c17e568 100644
- --- a/build.vc19/nfs_install/nfs_install.vcxproj
- +++ b/build.vc19/nfs_install/nfs_install.vcxproj
- @@ -89,6 +89,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>WIN32;_DEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -103,6 +104,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>_DEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -119,6 +121,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>WIN32;NDEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -137,6 +140,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>NDEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- diff --git a/build.vc19/nfs_mount/nfs_mount.vcxproj b/build.vc19/nfs_mount/nfs_mount.vcxproj
- index 8ecd741..efb2384 100644
- --- a/build.vc19/nfs_mount/nfs_mount.vcxproj
- +++ b/build.vc19/nfs_mount/nfs_mount.vcxproj
- @@ -89,6 +89,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>WIN32;_DEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -104,6 +105,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>_DEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -121,6 +123,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>WIN32;NDEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -140,6 +143,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>NDEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\sys;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- diff --git a/build.vc19/nfsd/nfsd.vcxproj b/build.vc19/nfsd/nfsd.vcxproj
- index 6f93e1d..ac73fbd 100644
- --- a/build.vc19/nfsd/nfsd.vcxproj
- +++ b/build.vc19/nfsd/nfsd.vcxproj
- @@ -89,6 +89,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>WIN32_LEAN_AND_MEAN;FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;STANDALONE_NFSD;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;WIN32;_DEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;..\..\sys;..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -104,6 +105,7 @@
- <Optimization>Disabled</Optimization>
- <PreprocessorDefinitions>WIN32_LEAN_AND_MEAN;FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;STANDALONE_NFSD;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;_DEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;..\..\sys;..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -121,6 +123,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>WIN32_LEAN_AND_MEAN;FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;WIN32;NDEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;..\..\sys;..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- @@ -140,6 +143,7 @@
- <IntrinsicFunctions>true</IntrinsicFunctions>
- <PreprocessorDefinitions>WIN32_LEAN_AND_MEAN;FD_SETSIZE=1024;INET6;NO_CB_4_KRB5P;_CRT_SECURE_NO_WARNINGS;_WINSOCK_DEPRECATED_NO_WARNINGS;NDEBUG;_CONSOLE;%(PreprocessorDefinitions)</PreprocessorDefinitions>
- <AdditionalIncludeDirectories>..\..\libtirpc\tirpc;..\..\sys;..\..\dll;%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
- + <LanguageStandard_C>stdc17</LanguageStandard_C>
- </ClCompile>
- <Link>
- <SubSystem>Console</SubSystem>
- diff --git a/daemon/nfs41_daemon.c b/daemon/nfs41_daemon.c
- index 31da803..0bb3299 100644
- --- a/daemon/nfs41_daemon.c
- +++ b/daemon/nfs41_daemon.c
- @@ -20,6 +20,10 @@
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- */
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- +
- #include <Windows.h>
- #include <process.h>
- #include <tchar.h>
- diff --git a/daemon/nfs41_daemon.h b/daemon/nfs41_daemon.h
- index f89b281..9801379 100644
- --- a/daemon/nfs41_daemon.h
- +++ b/daemon/nfs41_daemon.h
- @@ -23,6 +23,10 @@
- #ifndef __NFS41_DAEMON_H_
- #define __NFS41_DAEMON_H_ 1
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- +
- #include "nfs41_build_features.h"
- #include "idmap.h"
- diff --git a/dll/nfs41_np.c b/dll/nfs41_np.c
- index e194072..8fd75df 100644
- --- a/dll/nfs41_np.c
- +++ b/dll/nfs41_np.c
- @@ -19,6 +19,10 @@
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- */
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- +
- #include <windows.h>
- #include <npapi.h>
- #include <devioctl.h>
- diff --git a/install/nfs_install.c b/install/nfs_install.c
- index 3637657..c5fa45c 100644
- --- a/install/nfs_install.c
- +++ b/install/nfs_install.c
- @@ -26,6 +26,10 @@
- *
- */
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- +
- #include <tchar.h>
- #include "nfs41_driver.h"
- diff --git a/libtirpc/libtirpc/libtirpc.rc b/libtirpc/libtirpc/libtirpc.rc
- index 11c5d52..14ec524 100644
- --- a/libtirpc/libtirpc/libtirpc.rc
- +++ b/libtirpc/libtirpc/libtirpc.rc
- @@ -7,7 +7,7 @@
- //
- // Generated from the TEXTINCLUDE 2 resource.
- //
- -#include "afxres.h"
- +//#include "afxres.h"
- /////////////////////////////////////////////////////////////////////////////
- #undef APSTUDIO_READONLY_SYMBOLS
- diff --git a/libtirpc/src/gettimeofday.c b/libtirpc/src/gettimeofday.c
- index 5ccdfd4..4c8bef3 100644
- --- a/libtirpc/src/gettimeofday.c
- +++ b/libtirpc/src/gettimeofday.c
- @@ -1,6 +1,6 @@
- /* XXX NEED Copyright info */
- #include <wintirpc.h>
- -#include < time.h >
- +#include <time.h>
- #if defined(_MSC_VER) || defined(_MSC_EXTENSIONS)
- #define DELTA_EPOCH_IN_MICROSECS 11644473600000000Ui64
- diff --git a/libtirpc/tirpc/wintirpc.h b/libtirpc/tirpc/wintirpc.h
- index 88027c6..39f0cbd 100644
- --- a/libtirpc/tirpc/wintirpc.h
- +++ b/libtirpc/tirpc/wintirpc.h
- @@ -23,6 +23,11 @@
- #ifndef _TIRPC_WINTIRPC_H
- #define _TIRPC_WINTIRPC_H
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- +
- +
- /*
- * Eliminate warnings about possibly unsafe uses of snprintf and friends
- * XXX Think about cleaning these up and removing this later XXX
- diff --git a/mount/mount.c b/mount/mount.c
- index 093baa5..274fea7 100644
- --- a/mount/mount.c
- +++ b/mount/mount.c
- @@ -20,6 +20,9 @@
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- */
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- #include <crtdbg.h>
- #include <Windows.h>
- diff --git a/sys/nfs41_driver.c b/sys/nfs41_driver.c
- index be4a0d0..a831c37 100644
- --- a/sys/nfs41_driver.c
- +++ b/sys/nfs41_driver.c
- @@ -20,6 +20,14 @@
- * Inc., 51 Franklin Street, Fifth Floor, Boston, MA
- */
- +#ifndef _KERNEL_MODE
- +#error module requires kernel mode
- +#endif
- +
- +#if ((__STDC_VERSION__-0) < 201710L)
- +#error Code requires ISO C17
- +#endif
- +
- #define MINIRDR__NAME "Value is ignored, only fact of definition"
- #include <rx.h>
- #include <windef.h>
- --
- 2.42.1
- From 791e38ad40304e21eb992e4ec672906a32a0964a Mon Sep 17 00:00:00 2001
- From: Roland Mainz <roland.mainz@nrubsig.org>
- Date: Thu, 21 Dec 2023 19:00:17 +0100
- Subject: [PATCH 3/5] sys/nfs41_driver: Remove unneccesary debug output from
- kernel module
- Remove unneccesary debug output from kernel module
- Signed-off-by: Cedric Blancher <cedric.blancher@gmail.com>
- ---
- sys/nfs41_driver.c | 13 ++++---------
- 1 file changed, 4 insertions(+), 9 deletions(-)
- diff --git a/sys/nfs41_driver.c b/sys/nfs41_driver.c
- index a831c37..dd9f565 100644
- --- a/sys/nfs41_driver.c
- +++ b/sys/nfs41_driver.c
- @@ -5449,10 +5449,8 @@ NTSTATUS nfs41_QueryFileInformation(
- DbgEn();
- print_debug_filedirquery_header(RxContext);
- #endif
- - DbgP("--> nfs41_QueryFileInformation, RxContext->Info.LengthRemaining=%ld, "
- - "RxContext->LowIoContext.ParamsFor.IoCtl.InputBufferLength=%ld\n",
- - (long)RxContext->Info.LengthRemaining,
- - (long)RxContext->LowIoContext.ParamsFor.IoCtl.InputBufferLength);
- + DbgP("--> nfs41_QueryFileInformation, RxContext->Info.LengthRemaining=%ld\n",
- + (long)RxContext->Info.LengthRemaining);
- status = check_nfs41_dirquery_args(RxContext);
- if (status) {
- @@ -5462,11 +5460,8 @@ NTSTATUS nfs41_QueryFileInformation(
- RtlZeroMemory(RxContext->Info.Buffer, RxContext->Info.LengthRemaining);
- - DbgP("nfs41_QueryFileInformation, RxContext->Info.LengthRemaining=%ld, "
- - "RxContext->LowIoContext.ParamsFor.IoCtl.{InputBufferLength=%ld,OutputBufferLength=%ld}\n",
- - (long)RxContext->Info.LengthRemaining,
- - (long)RxContext->LowIoContext.ParamsFor.IoCtl.InputBufferLength,
- - (long)RxContext->LowIoContext.ParamsFor.IoCtl.OutputBufferLength);
- + DbgP("nfs41_QueryFileInformation, RxContext->Info.LengthRemaining=%ld\n",
- + (long)RxContext->Info.LengthRemaining);
- switch (InfoClass) {
- case FileEaInformation:
- --
- 2.42.1
- From ec8a40c578e34ac47f13ccada3fa4cf1702da68f Mon Sep 17 00:00:00 2001
- From: Roland Mainz <roland.mainz@nrubsig.org>
- Date: Thu, 21 Dec 2023 19:41:32 +0100
- Subject: [PATCH 4/5] sys/nfs41_debug: Fix pointer to int conversion for
- |PsGetCurrentProcessId()|
- Fix pointer to int conversion for |PsGetCurrentProcessId()|, it returns
- a |HANDLE|/(void *), but is used as |int| (printf %x) for debug output.
- The fix is to just use the pointer difference between ptr and 0.
- Signed-off-by: Cedric Blancher <cedric.blancher@gmail.com>
- ---
- sys/nfs41_debug.c | 4 ++--
- sys/nfs41_debug.h | 10 +++++++---
- 2 files changed, 9 insertions(+), 5 deletions(-)
- diff --git a/sys/nfs41_debug.c b/sys/nfs41_debug.c
- index d52d580..ade08d3 100644
- --- a/sys/nfs41_debug.c
- +++ b/sys/nfs41_debug.c
- @@ -55,7 +55,7 @@ ULONG __cdecl DbgP(IN PCCH fmt, ...)
- time_fields.Milliseconds, msg);
- #else
- DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_ERROR_LEVEL,
- - "[%04x] %s", PsGetCurrentProcessId(), msg);
- + "[%04x] %s", PsGetCurrentProcessShortDebugId(), msg);
- #endif
- }
- va_end(args);
- @@ -87,7 +87,7 @@ ULONG __cdecl print_error(IN PCCH fmt, ...)
- time_fields.Milliseconds, msg);
- #else
- DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_ERROR_LEVEL,
- - "[%04x] %s", PsGetCurrentProcessId(), msg);
- + "[%04x] %s", PsGetCurrentProcessShortDebugId(), msg);
- #endif
- }
- va_end(args);
- diff --git a/sys/nfs41_debug.h b/sys/nfs41_debug.h
- index d469e1f..917ec11 100644
- --- a/sys/nfs41_debug.h
- +++ b/sys/nfs41_debug.h
- @@ -56,19 +56,23 @@ void print_wait_status(int on, const char *str, NTSTATUS status,
- const char *opcode, PVOID entry, LONGLONG xid);
- void print_acl_args(SECURITY_INFORMATION info);
- +#define PTR2PTRDIFF_T(p) (((char *)(p))-((char *)0))
- +#define PsGetCurrentProcessShortDebugId() ((int)PTR2PTRDIFF_T(PsGetCurrentProcessId()))
- +
- +
- #define DbgEn() DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_ERROR_LEVEL, \
- - "--> [%s] [%04x] %s\n", _DRIVER_NAME_, PsGetCurrentProcessId(), \
- + "--> [%s] [%04x] %s\n", _DRIVER_NAME_, PsGetCurrentProcessShortDebugId(), \
- __FUNCTION__); try {
- #define DbgEx() DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_ERROR_LEVEL, \
- - "<-- [%s] [%04x] %s status = %08lx\n", _DRIVER_NAME_, PsGetCurrentProcessId(), \
- + "<-- [%s] [%04x] %s status = %08lx\n", _DRIVER_NAME_, PsGetCurrentProcessShortDebugId(), \
- __FUNCTION__, status); \
- } except (EXCEPTION_EXECUTE_HANDLER) { \
- status = GetExceptionCode() ; \
- DbgP("Exception encountered with value = Ox%x\n", status); \
- }
- #define DbgR() DbgPrintEx(DPFLTR_IHVNETWORK_ID, DPFLTR_ERROR_LEVEL, \
- - "<-- [%s] [%04x] %s\n", _DRIVER_NAME_, PsGetCurrentProcessId(), __FUNCTION__); \
- + "<-- [%s] [%04x] %s\n", _DRIVER_NAME_, PsGetCurrentProcessShortDebugId(), __FUNCTION__); \
- } except (EXCEPTION_EXECUTE_HANDLER) { \
- NTSTATUS status; \
- status = GetExceptionCode() ; \
- --
- 2.42.1
- From 15aabc04b2ad349df1ff0645f55c008b87f2ef50 Mon Sep 17 00:00:00 2001
- From: Roland Mainz <roland.mainz@nrubsig.org>
- Date: Fri, 22 Dec 2023 13:13:16 +0100
- Subject: [PATCH 5/5] mount: Fix "mount\mount.c(471): warning C6001: Using
- uninitialized memory 'dwLength'."
- Fix code analysis warning "mount\mount.c(471): warning C6001: Using
- uninitialized memory 'dwLength'."
- Signed-off-by: Cedric Blancher <cedric.blancher@gmail.com>
- ---
- mount/mount.c | 1 +
- 1 file changed, 1 insertion(+)
- diff --git a/mount/mount.c b/mount/mount.c
- index 274fea7..40eeb60 100644
- --- a/mount/mount.c
- +++ b/mount/mount.c
- @@ -468,6 +468,7 @@ static DWORD DoMount(
- goto out;
- /* fail if the connection already exists */
- + dwLength = NFS41_SYS_MAX_PATH_LEN;
- result = WNetGetConnection(pLocalName, (LPTSTR)szExisting, &dwLength);
- if (result == NO_ERROR)
- {
- --
- 2.42.1
msnfs41client: Patches for switch frrom legacy MSVC to ISO C17+analyser cleanup, 2023-12-22
Posted by Anonymous on Fri 22nd Dec 2023 13:56
raw | new post
Submit a correction or amendment below (click here to make a fresh posting)
After submitting an amendment, you'll be able to view the differences between the old and new posts easily.