This is an automated email from the ASF dual-hosted git repository.

vozerov pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git


The following commit(s) were added to refs/heads/master by this push:
     new aaa37fc  IGNITE-11227: SQL: Added missing security checks which were 
incorrectly removed in IGNITE-11227.
aaa37fc is described below

commit aaa37fc4d25e691200fdbc5bd5adb73941f099c1
Author: devozerov <[email protected]>
AuthorDate: Mon Mar 11 12:13:01 2019 +0300

    IGNITE-11227: SQL: Added missing security checks which were incorrectly 
removed in IGNITE-11227.
---
 .../ignite/internal/processors/query/h2/IgniteH2Indexing.java       | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/IgniteH2Indexing.java
 
b/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/IgniteH2Indexing.java
index 17e9b1d..5d912aa 100644
--- 
a/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/IgniteH2Indexing.java
+++ 
b/modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/IgniteH2Indexing.java
@@ -456,9 +456,6 @@ public class IgniteH2Indexing implements GridQueryIndexing {
                 throw new IgniteSQLException("SELECT FOR UPDATE query requires 
transactional " +
                     "cache with MVCC enabled.", 
IgniteQueryErrorCode.UNSUPPORTED_OPERATION);
 
-            if (ctx.security().enabled())
-                checkSecurity(select.cacheIds());
-
             GridNearTxSelectForUpdateFuture sfuFut = null;
 
             int opTimeout = qryParams.timeout();
@@ -1426,6 +1423,9 @@ public class IgniteH2Indexing implements 
GridQueryIndexing {
         MvccQueryTracker mvccTracker,
         GridQueryCancel cancel
     ) throws IgniteCheckedException {
+        if (ctx.security().enabled())
+            checkSecurity(select.cacheIds());
+
         boolean autoStartTx = mvccEnabled(ctx) && !qryParams.autoCommit() && 
tx(ctx) == null;
 
         Iterable<List<?>> iter;

Reply via email to