common: Print octal mode correctly (#910)

jepio · web-flow · commit a60ebbf1c3e0 · 2025-03-18T10:26:05.000-07:00
After the codebase was unified to handling mode bits as octal, the printf specifiers were not updated to handle that. Fix this.
Signed-off-by: Jeremi Piotrowski &lt;jpiotrowski@microsoft.com&gt;
diff --git a/src/common/commonutils/FileUtils.c b/src/common/commonutils/FileUtils.c
@@ -195,7 +195,7 @@ static bool InternalSecureSaveToFile(const char* fileName, const char* mode, con
     char* fileContents = NULL;
     unsigned int ownerId = 0;
     unsigned int groupId = 0;
-    unsigned int access = 644;
+    unsigned int access = 0644;
     int status = 0;
     bool result = false;
 
@@ -219,7 +219,7 @@ static bool InternalSecureSaveToFile(const char* fileName, const char* mode, con
     {
         if (0 == GetDirectoryAccess(fileDirectory, &ownerId, &groupId, &access, log))
         {
-            OsConfigLogInfo(log, "InternalSecureSaveToFile: directory '%s' exists, is owned by user (%u, %u) and has access mode %u",
+            OsConfigLogInfo(log, "InternalSecureSaveToFile: directory '%s' exists, is owned by user (%u, %u) and has access mode %03o",
                 fileDirectory, ownerId, groupId, access);
         }
     }
@@ -606,19 +606,19 @@ static int CheckAccess(bool directory, const char* name, int desiredOwnerId, int
 
                 if (currentMode != desiredMode)
                 {
-                    OsConfigLogInfo(log, "CheckAccess: access to '%s' (%d) does not match expected (%d)", name, currentMode, desiredMode);
-                    OsConfigCaptureReason(reason, "Access to '%s' (%d) does not match expected (%d)", name, currentMode, desiredMode);
+                    OsConfigLogInfo(log, "CheckAccess: access to '%s' (%03o) does not match expected (%03o)", name, currentMode, desiredMode);
+                    OsConfigCaptureReason(reason, "Access to '%s' (%03o) does not match expected (%03o)", name, currentMode, desiredMode);
                     result = ENOENT;
                 }
                 else
                 {
                     // Special case for the MPI Client
                     if (NULL != log)
                     {
-                        OsConfigLogInfo(log, "CheckAccess: access to '%s' (%d) matches expected (%d)", name, currentMode, desiredMode);
+                        OsConfigLogInfo(log, "CheckAccess: access to '%s' (%03o) matches expected (%03o)", name, currentMode, desiredMode);
                     }
 
-                    OsConfigCaptureSuccessReason(reason, "'%s' has required access (%d) and ownership (uid: %d, gid: %u)", name, desiredMode, desiredOwnerId, desiredGroupId);
+                    OsConfigCaptureSuccessReason(reason, "'%s' has required access (%03o) and ownership (uid: %d, gid: %u)", name, desiredMode, desiredOwnerId, desiredGroupId);
                     result = 0;
                 }
             }
@@ -659,7 +659,7 @@ static int SetAccess(bool directory, const char* name, unsigned int desiredOwner
     {
         if (0 == (result = CheckAccess(directory, name, desiredOwnerId, desiredGroupId, desiredAccess, false, NULL, log)))
         {
-            OsConfigLogInfo(log, "SetAccess: desired '%s' ownership (owner %u, group %u with access %u) already set",
+            OsConfigLogInfo(log, "SetAccess: desired '%s' ownership (owner %u, group %u with access %03o) already set",
                 name, desiredOwnerId, desiredGroupId, desiredAccess);
             result = 0;
         }
@@ -671,12 +671,12 @@ static int SetAccess(bool directory, const char* name, unsigned int desiredOwner
 
                 if (0 == (result = chmod(name, desiredAccess)))
                 {
-                    OsConfigLogInfo(log, "SetAccess: successfully set access to '%s' to %u", name, desiredAccess);
+                    OsConfigLogInfo(log, "SetAccess: successfully set access to '%s' to %03o", name, desiredAccess);
                 }
                 else
                 {
                     result = errno ? errno : ENOENT;
-                    OsConfigLogInfo(log, "SetAccess: 'chmod %d %s' failed with %d", desiredAccess, name, result);
+                    OsConfigLogInfo(log, "SetAccess: 'chmod %03o %s' failed with %d", desiredAccess, name, result);
                 }
             }
             else
@@ -922,7 +922,7 @@ int RenameFileWithOwnerAndAccess(const char* original, const char* target, OsCon
         }
         else
         {
-            OsConfigLogDebug(log, "RenameFileWithOwnerAndAccess: '%s' renamed to '%s' with restored original owner %u, group %u and access mode %u",
+            OsConfigLogDebug(log, "RenameFileWithOwnerAndAccess: '%s' renamed to '%s' with restored original owner %u, group %u and access mode %03o",
                 original, target, ownerId, groupId, mode);
         }
 
@@ -1920,11 +1920,11 @@ int DisablePostfixNetworkListening(OsConfigLogHandle log)
         OsConfigLogInfo(log, "DisablePostfixNetworkListening: directory '%s' does not exist", etcPostfix);
         if (0 == (status = mkdir(etcPostfix, mode)))
         {
-            OsConfigLogInfo(log, "DisablePostfixNetworkListening: created directory '%s' with %d access", etcPostfix, mode);
+            OsConfigLogInfo(log, "DisablePostfixNetworkListening: created directory '%s' with %03o access", etcPostfix, mode);
         }
         else
         {
-            OsConfigLogInfo(log, "DisablePostfixNetworkListening: cannot create directory '%s' with %d access (%d)", etcPostfix, mode, errno);
+            OsConfigLogInfo(log, "DisablePostfixNetworkListening: cannot create directory '%s' with %d access (%03o)", etcPostfix, mode, errno);
         }
     }
 
diff --git a/src/common/commonutils/SshUtils.c b/src/common/commonutils/SshUtils.c
@@ -826,7 +826,7 @@ static int SetSshWarningBanner(unsigned int desiredBannerFileAccess, const char*
         {
             if (0 != (status = SetFileAccess(g_sshBannerFile, 0, 0, desiredBannerFileAccess, log)))
             {
-                OsConfigLogInfo(log, "SetSshWarningBanner: failed to set desired access %u on banner file %s (%d)", desiredBannerFileAccess, g_sshBannerFile, status);
+                OsConfigLogInfo(log, "SetSshWarningBanner: failed to set desired access %03o on banner file %s (%d)", desiredBannerFileAccess, g_sshBannerFile, status);
             }
         }
         else
diff --git a/src/common/commonutils/UserUtils.c b/src/common/commonutils/UserUtils.c
@@ -1734,7 +1734,7 @@ int CheckRestrictedUserHomeDirectories(unsigned int* modes, unsigned int numberO
                 {
                     if (0 == CheckDirectoryAccess(userList[i].home, userList[i].userId, userList[i].groupId, modes[j], true, NULL, log))
                     {
-                        OsConfigLogInfo(log, "CheckRestrictedUserHomeDirectories: user '%s' (%u, %u) has proper restricted access (%u) for their assigned home directory '%s'",
+                        OsConfigLogInfo(log, "CheckRestrictedUserHomeDirectories: user '%s' (%u, %u) has proper restricted access (%03o) for their assigned home directory '%s'",
                             userList[i].username, userList[i].userId, userList[i].groupId, modes[j], userList[i].home);
                         oneGoodMode = true;
                         break;
@@ -1797,7 +1797,7 @@ int SetRestrictedUserHomeDirectories(unsigned int* modes, unsigned int numberOfM
                 {
                     if (0 == CheckDirectoryAccess(userList[i].home, userList[i].userId, userList[i].groupId, modes[j], true, NULL, log))
                     {
-                        OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) already has proper restricted access (%u) for their assigned home directory '%s'",
+                        OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) already has proper restricted access (%03o) for their assigned home directory '%s'",
                             userList[i].username, userList[i].userId, userList[i].groupId, modes[j], userList[i].home);
                         oneGoodMode = true;
                         break;
@@ -1808,12 +1808,12 @@ int SetRestrictedUserHomeDirectories(unsigned int* modes, unsigned int numberOfM
                 {
                     if (0 == (_status = SetDirectoryAccess(userList[i].home, userList[i].userId, userList[i].groupId, userList[i].isRoot ? modeForRoot : modeForOthers, log)))
                     {
-                        OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) has now proper restricted access (%u) for their assigned home directory '%s'",
+                        OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) has now proper restricted access (%03o) for their assigned home directory '%s'",
                             userList[i].username, userList[i].userId, userList[i].groupId, userList[i].isRoot ? modeForRoot : modeForOthers, userList[i].home);
                     }
                     else
                     {
-                        OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: cannot set restricted access (%u) for user '%s' (%u, %u) assigned home directory '%s' (%d)",
+                        OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: cannot set restricted access (%03o) for user '%s' (%u, %u) assigned home directory '%s' (%d)",
                             userList[i].isRoot ? modeForRoot : modeForOthers, userList[i].username, userList[i].userId, userList[i].groupId, userList[i].home, _status);
 
                         if (0 == status)
@@ -2877,7 +2877,7 @@ int CheckUsersRestrictedDotFiles(unsigned int* modes, unsigned int numberOfModes
                         {
                             if (0 == CheckFileAccess(path, userList[i].userId, userList[i].groupId, modes[j], NULL, log))
                             {
-                                OsConfigLogInfo(log, "CheckUsersRestrictedDotFiles: user '%s' (%u, %u) has proper restricted access (%u) for their dot file '%s'",
+                                OsConfigLogInfo(log, "CheckUsersRestrictedDotFiles: user '%s' (%u, %u) has proper restricted access (%03o) for their dot file '%s'",
                                     userList[i].username, userList[i].userId, userList[i].groupId, modes[j], path);
                                 oneGoodMode = true;
                                 break;
@@ -2967,7 +2967,7 @@ int SetUsersRestrictedDotFiles(unsigned int* modes, unsigned int numberOfModes,
                         {
                             if (0 == CheckFileAccess(path, userList[i].userId, userList[i].groupId, modes[j], NULL, log))
                             {
-                                OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) already has proper restricted access (%u) set for their dot file '%s'",
+                                OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) already has proper restricted access (%03o) set for their dot file '%s'",
                                     userList[i].username, userList[i].userId, userList[i].groupId, modes[j], path);
                                 oneGoodMode = true;
                                 break;
@@ -2978,7 +2978,7 @@ int SetUsersRestrictedDotFiles(unsigned int* modes, unsigned int numberOfModes,
                         {
                             if (0 == (_status = SetFileAccess(path, userList[i].userId, userList[i].groupId, mode, log)))
                             {
-                                OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) now has restricted access (%u) set for their dot file '%s'",
+                                OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) now has restricted access (%03o) set for their dot file '%s'",
                                     userList[i].username, userList[i].userId, userList[i].groupId, mode, path);
                             }
                             else

Original file line number	Diff line number	Diff line change
`@@ -195,7 +195,7 @@ static bool InternalSecureSaveToFile(const char* fileName, const char* mode, con`
`195`	`195`	`char* fileContents = NULL;`
`196`	`196`	`unsigned int ownerId = 0;`
`197`	`197`	`unsigned int groupId = 0;`
`198`		`- unsigned int access = 644;`
	`198`	`+ unsigned int access = 0644;`
`199`	`199`	`int status = 0;`
`200`	`200`	`bool result = false;`
`201`	`201`
`@@ -219,7 +219,7 @@ static bool InternalSecureSaveToFile(const char* fileName, const char* mode, con`
`219`	`219`	`{`
`220`	`220`	`if (0 == GetDirectoryAccess(fileDirectory, &ownerId, &groupId, &access, log))`
`221`	`221`	`{`
`222`		`- OsConfigLogInfo(log, "InternalSecureSaveToFile: directory '%s' exists, is owned by user (%u, %u) and has access mode %u",`
	`222`	`+ OsConfigLogInfo(log, "InternalSecureSaveToFile: directory '%s' exists, is owned by user (%u, %u) and has access mode %03o",`
`223`	`223`	`fileDirectory, ownerId, groupId, access);`
`224`	`224`	`}`
`225`	`225`	`}`
`@@ -606,19 +606,19 @@ static int CheckAccess(bool directory, const char* name, int desiredOwnerId, int`
`606`	`606`
`607`	`607`	`if (currentMode != desiredMode)`
`608`	`608`	`{`
`609`		`- OsConfigLogInfo(log, "CheckAccess: access to '%s' (%d) does not match expected (%d)", name, currentMode, desiredMode);`
`610`		`- OsConfigCaptureReason(reason, "Access to '%s' (%d) does not match expected (%d)", name, currentMode, desiredMode);`
	`609`	`+ OsConfigLogInfo(log, "CheckAccess: access to '%s' (%03o) does not match expected (%03o)", name, currentMode, desiredMode);`
	`610`	`+ OsConfigCaptureReason(reason, "Access to '%s' (%03o) does not match expected (%03o)", name, currentMode, desiredMode);`
`611`	`611`	`result = ENOENT;`
`612`	`612`	`}`
`613`	`613`	`else`
`614`	`614`	`{`
`615`	`615`	`// Special case for the MPI Client`
`616`	`616`	`if (NULL != log)`
`617`	`617`	`{`
`618`		`- OsConfigLogInfo(log, "CheckAccess: access to '%s' (%d) matches expected (%d)", name, currentMode, desiredMode);`
	`618`	`+ OsConfigLogInfo(log, "CheckAccess: access to '%s' (%03o) matches expected (%03o)", name, currentMode, desiredMode);`
`619`	`619`	`}`
`620`	`620`
`621`		`- OsConfigCaptureSuccessReason(reason, "'%s' has required access (%d) and ownership (uid: %d, gid: %u)", name, desiredMode, desiredOwnerId, desiredGroupId);`
	`621`	`+ OsConfigCaptureSuccessReason(reason, "'%s' has required access (%03o) and ownership (uid: %d, gid: %u)", name, desiredMode, desiredOwnerId, desiredGroupId);`
`622`	`622`	`result = 0;`
`623`	`623`	`}`
`624`	`624`	`}`
`@@ -659,7 +659,7 @@ static int SetAccess(bool directory, const char* name, unsigned int desiredOwner`
`659`	`659`	`{`
`660`	`660`	`if (0 == (result = CheckAccess(directory, name, desiredOwnerId, desiredGroupId, desiredAccess, false, NULL, log)))`
`661`	`661`	`{`
`662`		`- OsConfigLogInfo(log, "SetAccess: desired '%s' ownership (owner %u, group %u with access %u) already set",`
	`662`	`+ OsConfigLogInfo(log, "SetAccess: desired '%s' ownership (owner %u, group %u with access %03o) already set",`
`663`	`663`	`name, desiredOwnerId, desiredGroupId, desiredAccess);`
`664`	`664`	`result = 0;`
`665`	`665`	`}`
`@@ -671,12 +671,12 @@ static int SetAccess(bool directory, const char* name, unsigned int desiredOwner`
`671`	`671`
`672`	`672`	`if (0 == (result = chmod(name, desiredAccess)))`
`673`	`673`	`{`
`674`		`- OsConfigLogInfo(log, "SetAccess: successfully set access to '%s' to %u", name, desiredAccess);`
	`674`	`+ OsConfigLogInfo(log, "SetAccess: successfully set access to '%s' to %03o", name, desiredAccess);`
`675`	`675`	`}`
`676`	`676`	`else`
`677`	`677`	`{`
`678`	`678`	`result = errno ? errno : ENOENT;`
`679`		`- OsConfigLogInfo(log, "SetAccess: 'chmod %d %s' failed with %d", desiredAccess, name, result);`
	`679`	`+ OsConfigLogInfo(log, "SetAccess: 'chmod %03o %s' failed with %d", desiredAccess, name, result);`
`680`	`680`	`}`
`681`	`681`	`}`
`682`	`682`	`else`
`@@ -922,7 +922,7 @@ int RenameFileWithOwnerAndAccess(const char* original, const char* target, OsCon`
`922`	`922`	`}`
`923`	`923`	`else`
`924`	`924`	`{`
`925`		`- OsConfigLogDebug(log, "RenameFileWithOwnerAndAccess: '%s' renamed to '%s' with restored original owner %u, group %u and access mode %u",`
	`925`	`+ OsConfigLogDebug(log, "RenameFileWithOwnerAndAccess: '%s' renamed to '%s' with restored original owner %u, group %u and access mode %03o",`
`926`	`926`	`original, target, ownerId, groupId, mode);`
`927`	`927`	`}`
`928`	`928`
`@@ -1920,11 +1920,11 @@ int DisablePostfixNetworkListening(OsConfigLogHandle log)`
`1920`	`1920`	`OsConfigLogInfo(log, "DisablePostfixNetworkListening: directory '%s' does not exist", etcPostfix);`
`1921`	`1921`	`if (0 == (status = mkdir(etcPostfix, mode)))`
`1922`	`1922`	`{`
`1923`		`- OsConfigLogInfo(log, "DisablePostfixNetworkListening: created directory '%s' with %d access", etcPostfix, mode);`
	`1923`	`+ OsConfigLogInfo(log, "DisablePostfixNetworkListening: created directory '%s' with %03o access", etcPostfix, mode);`
`1924`	`1924`	`}`
`1925`	`1925`	`else`
`1926`	`1926`	`{`
`1927`		`- OsConfigLogInfo(log, "DisablePostfixNetworkListening: cannot create directory '%s' with %d access (%d)", etcPostfix, mode, errno);`
	`1927`	`+ OsConfigLogInfo(log, "DisablePostfixNetworkListening: cannot create directory '%s' with %d access (%03o)", etcPostfix, mode, errno);`
`1928`	`1928`	`}`
`1929`	`1929`	`}`
`1930`	`1930`
Original file line number	Diff line number	Diff line change
`@@ -826,7 +826,7 @@ static int SetSshWarningBanner(unsigned int desiredBannerFileAccess, const char*`
`826`	`826`	`{`
`827`	`827`	`if (0 != (status = SetFileAccess(g_sshBannerFile, 0, 0, desiredBannerFileAccess, log)))`
`828`	`828`	`{`
`829`		`- OsConfigLogInfo(log, "SetSshWarningBanner: failed to set desired access %u on banner file %s (%d)", desiredBannerFileAccess, g_sshBannerFile, status);`
	`829`	`+ OsConfigLogInfo(log, "SetSshWarningBanner: failed to set desired access %03o on banner file %s (%d)", desiredBannerFileAccess, g_sshBannerFile, status);`
`830`	`830`	`}`
`831`	`831`	`}`
`832`	`832`	`else`
Original file line number	Diff line number	Diff line change
`@@ -1734,7 +1734,7 @@ int CheckRestrictedUserHomeDirectories(unsigned int* modes, unsigned int numberO`
`1734`	`1734`	`{`
`1735`	`1735`	`if (0 == CheckDirectoryAccess(userList[i].home, userList[i].userId, userList[i].groupId, modes[j], true, NULL, log))`
`1736`	`1736`	`{`
`1737`		`- OsConfigLogInfo(log, "CheckRestrictedUserHomeDirectories: user '%s' (%u, %u) has proper restricted access (%u) for their assigned home directory '%s'",`
	`1737`	`+ OsConfigLogInfo(log, "CheckRestrictedUserHomeDirectories: user '%s' (%u, %u) has proper restricted access (%03o) for their assigned home directory '%s'",`
`1738`	`1738`	`userList[i].username, userList[i].userId, userList[i].groupId, modes[j], userList[i].home);`
`1739`	`1739`	`oneGoodMode = true;`
`1740`	`1740`	`break;`
`@@ -1797,7 +1797,7 @@ int SetRestrictedUserHomeDirectories(unsigned int* modes, unsigned int numberOfM`
`1797`	`1797`	`{`
`1798`	`1798`	`if (0 == CheckDirectoryAccess(userList[i].home, userList[i].userId, userList[i].groupId, modes[j], true, NULL, log))`
`1799`	`1799`	`{`
`1800`		`- OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) already has proper restricted access (%u) for their assigned home directory '%s'",`
	`1800`	`+ OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) already has proper restricted access (%03o) for their assigned home directory '%s'",`
`1801`	`1801`	`userList[i].username, userList[i].userId, userList[i].groupId, modes[j], userList[i].home);`
`1802`	`1802`	`oneGoodMode = true;`
`1803`	`1803`	`break;`
`@@ -1808,12 +1808,12 @@ int SetRestrictedUserHomeDirectories(unsigned int* modes, unsigned int numberOfM`
`1808`	`1808`	`{`
`1809`	`1809`	`if (0 == (_status = SetDirectoryAccess(userList[i].home, userList[i].userId, userList[i].groupId, userList[i].isRoot ? modeForRoot : modeForOthers, log)))`
`1810`	`1810`	`{`
`1811`		`- OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) has now proper restricted access (%u) for their assigned home directory '%s'",`
	`1811`	`+ OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: user '%s' (%u, %u) has now proper restricted access (%03o) for their assigned home directory '%s'",`
`1812`	`1812`	`userList[i].username, userList[i].userId, userList[i].groupId, userList[i].isRoot ? modeForRoot : modeForOthers, userList[i].home);`
`1813`	`1813`	`}`
`1814`	`1814`	`else`
`1815`	`1815`	`{`
`1816`		`- OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: cannot set restricted access (%u) for user '%s' (%u, %u) assigned home directory '%s' (%d)",`
	`1816`	`+ OsConfigLogInfo(log, "SetRestrictedUserHomeDirectories: cannot set restricted access (%03o) for user '%s' (%u, %u) assigned home directory '%s' (%d)",`
`1817`	`1817`	`userList[i].isRoot ? modeForRoot : modeForOthers, userList[i].username, userList[i].userId, userList[i].groupId, userList[i].home, _status);`
`1818`	`1818`
`1819`	`1819`	`if (0 == status)`
`@@ -2877,7 +2877,7 @@ int CheckUsersRestrictedDotFiles(unsigned int* modes, unsigned int numberOfModes`
`2877`	`2877`	`{`
`2878`	`2878`	`if (0 == CheckFileAccess(path, userList[i].userId, userList[i].groupId, modes[j], NULL, log))`
`2879`	`2879`	`{`
`2880`		`- OsConfigLogInfo(log, "CheckUsersRestrictedDotFiles: user '%s' (%u, %u) has proper restricted access (%u) for their dot file '%s'",`
	`2880`	`+ OsConfigLogInfo(log, "CheckUsersRestrictedDotFiles: user '%s' (%u, %u) has proper restricted access (%03o) for their dot file '%s'",`
`2881`	`2881`	`userList[i].username, userList[i].userId, userList[i].groupId, modes[j], path);`
`2882`	`2882`	`oneGoodMode = true;`
`2883`	`2883`	`break;`
`@@ -2967,7 +2967,7 @@ int SetUsersRestrictedDotFiles(unsigned int* modes, unsigned int numberOfModes,`
`2967`	`2967`	`{`
`2968`	`2968`	`if (0 == CheckFileAccess(path, userList[i].userId, userList[i].groupId, modes[j], NULL, log))`
`2969`	`2969`	`{`
`2970`		`- OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) already has proper restricted access (%u) set for their dot file '%s'",`
	`2970`	`+ OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) already has proper restricted access (%03o) set for their dot file '%s'",`
`2971`	`2971`	`userList[i].username, userList[i].userId, userList[i].groupId, modes[j], path);`
`2972`	`2972`	`oneGoodMode = true;`
`2973`	`2973`	`break;`
`@@ -2978,7 +2978,7 @@ int SetUsersRestrictedDotFiles(unsigned int* modes, unsigned int numberOfModes,`
`2978`	`2978`	`{`
`2979`	`2979`	`if (0 == (_status = SetFileAccess(path, userList[i].userId, userList[i].groupId, mode, log)))`
`2980`	`2980`	`{`
`2981`		`- OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) now has restricted access (%u) set for their dot file '%s'",`
	`2981`	`+ OsConfigLogInfo(log, "SetUsersRestrictedDotFiles: user '%s' (%u, %u) now has restricted access (%03o) set for their dot file '%s'",`
`2982`	`2982`	`userList[i].username, userList[i].userId, userList[i].groupId, mode, path);`
`2983`	`2983`	`}`
`2984`	`2984`	`else`