[ 
https://issues.apache.org/jira/browse/HADOOP-10356?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14536435#comment-14536435
 ] 

Hudson commented on HADOOP-10356:
---------------------------------

SUCCESS: Integrated in Hadoop-Yarn-trunk #922 (See 
[https://builds.apache.org/job/Hadoop-Yarn-trunk/922/])
HADOOP-10356. Corrections in winutils/chmod.c (Contributed by René 
Nyffenegger) (arp: rev a0fae8dcae9e10cc7b751ed216017ded29177be1)
* hadoop-common-project/hadoop-common/CHANGES.txt
* hadoop-common-project/hadoop-common/src/main/winutils/chmod.c


> Corrections in winutils/chmod.c
> -------------------------------
>
>                 Key: HADOOP-10356
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10356
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: bin
>         Environment: Windows
>            Reporter: René Nyffenegger
>            Assignee: René Nyffenegger
>            Priority: Trivial
>              Labels: BB2015-05-RFC
>             Fix For: 2.8.0
>
>         Attachments: HADOOP-10356.patch
>
>
> There are two small things in winutils/chmod.c:
>   pathName should be pointer to a constant WSTR
>   the declartion
>       LPWSTR pathName = NULL;
>   seems to be wrong.
>         LPCWSTR pathName = NULL;
>    should b used instead.
>   ------------------------------------------------
>    I believe the fragment
>       switch (c)
>       {
>       case NULL:
>    to be wrong as pointers are not permitted as
>    case values.
>   ------------------------------------------------



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to