[ 
https://issues.apache.org/jira/browse/IMPALA-15382?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aleksandr Efimov updated IMPALA-15382:
--------------------------------------
    Description: RequestPoolService::ResolveRequestPool() stores the status 
returned by JniUtil::CallJniMethod() but does not check it before reading the 
result. If the Java call throws while converting a user principal or resolving 
a pool, the default result is read instead. The client can then receive a 
misleading denied-access error with an empty resolved pool, while the original 
JNI error is lost. Return the JNI status before inspecting the result and add a 
backend regression test that verifies a short-name conversion error is 
propagated.  (was: RequestPoolService::ResolveRequestPool() stores the status 
returned by JniUtil::CallJniMethod() but does not check it before reading the 
result.

If the Java call throws while converting a user principal or resolving a pool, 
the default result is read instead. The client can then receive a misleading 
denied-access error with an empty resolved pool, while the original JNI error 
is lost.

Return the JNI status before inspecting the result and add a backend regression 
test that verifies a short-name conversion error is propagated.)

> Request pool resolution ignores JNI errors
> ------------------------------------------
>
>                 Key: IMPALA-15382
>                 URL: https://issues.apache.org/jira/browse/IMPALA-15382
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Backend
>            Reporter: Aleksandr Efimov
>            Priority: Major
>
> RequestPoolService::ResolveRequestPool() stores the status returned by 
> JniUtil::CallJniMethod() but does not check it before reading the result. If 
> the Java call throws while converting a user principal or resolving a pool, 
> the default result is read instead. The client can then receive a misleading 
> denied-access error with an empty resolved pool, while the original JNI error 
> is lost. Return the JNI status before inspecting the result and add a backend 
> regression test that verifies a short-name conversion error is propagated.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to