HADOOP-10356. Corrections in winutils/chmod.c (Contributed by René Nyffenegger)

hadoop-common-project/hadoop-common/CHANGES.txt

@@ -179,6 +179,9 @@ Release 2.8.0 - UNRELEASED
     HADOOP-11877. SnappyDecompressor's Logger class name is wrong
     (surendra singh lilhore via vinayakumarb)
 
+    HADOOP-10356. Corrections in winutils/chmod.c. (Rene Nyffenegger via
+    Arpit Agarwal)
+
 Release 2.7.1 - UNRELEASED
 
   INCOMPATIBLE CHANGES

hadoop-common-project/hadoop-common/src/main/winutils/chmod.c

@@ -105,7 +105,7 @@ static BOOL ChangeFileModeRecursively(__in LPCWSTR path, __in_opt INT mode,
 //
 int Chmod(__in int argc, __in_ecount(argc) wchar_t *argv[])
 {
-  LPWSTR pathName = NULL;
+  LPCWSTR pathName = NULL;
   LPWSTR longPathName = NULL;
 
   BOOL recursive = FALSE;
@@ -128,7 +128,7 @@ int Chmod(__in int argc, __in_ecount(argc) wchar_t *argv[])
     return EXIT_FAILURE;
   }
 
-  // Convert the path the the long path
+  // Convert the path to the long path
   //
   dwRtnCode = ConvertToLongPath(pathName, &longPathName);
   if (dwRtnCode != ERROR_SUCCESS)
@@ -763,7 +763,7 @@ static BOOL ParseMode(LPCWSTR modeString, PMODE_CHANGE_ACTION *pActions)
     {
       switch (c)
       {
-      case NULL:
+      case L'\0':
         __fallthrough;
       case L',':
         i++;