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

Vadim Spector commented on SENTRY-1993:
---------------------------------------

I provided a patch with two extra lines of code in test class, to directly test 
Misha's fix. Unless there are any objectives, I'll proceed with the commit. It 
will require reverting previous commit SENTRY-1993 commit.

> StringIndexOutOfBoundsException in HMSPathsDumper.java
> ------------------------------------------------------
>
>                 Key: SENTRY-1993
>                 URL: https://issues.apache.org/jira/browse/SENTRY-1993
>             Project: Sentry
>          Issue Type: Bug
>            Reporter: Misha Dmitriev
>            Assignee: Misha Dmitriev
>         Attachments: SENTRY-1993.01.patch, SENTRY-1993.02.patch
>
>
> The following line in HMSPathsDumper.java is causing 
> StringIndexOutOfBoundsException:
> {code}
> if (tChildPathElement.charAt(0) == DupDetector.REPLACEMENT_STRING_PREFIX) {
> {code}
> It only happens when a path element is "", when someone mistakenly specifies 
> hdfs path with two "/" in the path section, like 
> hdfs://server//element1//element2, instead of 
> hdfs://server/element1/element2. In principle, such paths are invalid, but 
> this code should be made resistant to them anyway.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to