zhangshuyan0 opened a new pull request, #6067:
URL: https://github.com/apache/hadoop/pull/6067

   ### Description of PR
   When we need to use OIV to print EC information for a directory, 
`PBImageTextWriter#getErasureCodingPolicyName` will be called. Currently, this 
method uses `XATTR_ERASURECODING_POLICY.contains(xattr.getName())` to filter 
and obtain EC XAttr, which is very dangerous. If we have a XAttr whose name 
happens to be a substring of `hdfs.erasurecoding.policy`, then 
`getErasureCodingPolicyName` will return the wrong result. Our internal 
production environment has customized some XAttrs, and this bug caused errors 
in the parsing results of OIV when using `-ec` option. 
   
   ### How was this patch tested?
   Add a new UT.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to