sentry: authorization.go:129: cannot use CheckPrivilege without a txn (1) assertion failure Wraps: (2) attached stack trace -- stack trace: | github.com/cockroachdb/cockroach/pkg/sql.(*planner).CheckPrivi... #82942

cockroach-teamcity · 2022-06-15T15:55:12Z

This issue was autofiled by Sentry. It represents a crash or reported error on a live cluster with telemetry enabled.

Sentry link: https://sentry.io/organizations/cockroach-labs/issues/3351786350/?referrer=webhooks_plugin

Panic message:

authorization.go:129: cannot use CheckPrivilege without a txn
(1) assertion failure
Wraps: (2) attached stack trace
-- stack trace:
| github.com/cockroachdb/cockroach/pkg/sql.(*planner).CheckPrivilegeForUser
| github.com/cockroachdb/cockroach/pkg/sql/authorization.go:129
| github.com/cockroachdb/cockroach/pkg/sql.(*planner).canResolveDescUnderSchema
| github.com/cockroachdb/cockroach/pkg/sql/authorization.go:743
| github.com/cockroachdb/cockroach/pkg/sql.(*optCatalog).ResolveDataSource
| github.com/cockroachdb/cockroach/pkg/sql/opt_catalog.go:214
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).resolveDataSource
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/util.go:606
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildDataSource
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:104
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildDataSource
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:60
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildFromTablesRightDeep
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:1155
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildFromTables
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:1132
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildFrom
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:1059
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildSelectClause
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:980
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildSelectStmtWithoutParens
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:928
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildSelect.func1
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:901
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).processWiths
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/with.go:115
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildSelect
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:900
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildStmt
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/builder.go:281
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildCTE
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/with.go:194
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildCTEs
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/with.go:139
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).processWiths
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/with.go:112
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildSelect
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/select.go:900
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildStmt
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/builder.go:281
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).buildStmtAtRoot
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/builder.go:241
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder.(*Builder).Build
| github.com/cockroachdb/cockroach/pkg/sql/opt/optbuilder/builder.go:215
| github.com/cockroachdb/cockroach/pkg/sql.(*optPlanningCtx).buildReusableMemo
| github.com/cockroachdb/cockroach/pkg/sql/plan_opt.go:399
| github.com/cockroachdb/cockroach/pkg/sql.(*planner).prepareUsingOptimizer
| github.com/cockroachdb/cockroach/pkg/sql/plan_opt.go:142
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).populatePrepared
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_prepare.go:278
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).prepare.func1
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_prepare.go:235
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).prepare
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_prepare.go:240
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).addPreparedStmt
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_prepare.go:102
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execPrepare
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor_prepare.go:63
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).execCmd
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1975
| github.com/cockroachdb/cockroach/pkg/sql.(*connExecutor).run
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:1799
| github.com/cockroachdb/cockroach/pkg/sql.(*Server).ServeConn
| github.com/cockroachdb/cockroach/pkg/sql/conn_executor.go:747
Wraps: (3) cannot use CheckPrivilege without a txn
Error types: (1) *assert.withAssertionFailure (2) *withstack.withStack (3) *errutil.leafError
-- report composition:
*errutil.leafError: cannot use CheckPrivilege without a txn
authorization.go:129: *withstack.withStack (top exception)
*assert.withAssertionFailure

Stacktrace (expand for inline code snippets):

cockroach/pkg/sql/authorization.go

Lines 128 to 130 in 5b78463

    
           if p.txn == nil || !p.txn.IsOpen() { 
        
           	return errors.AssertionFailedf("cannot use CheckPrivilege without a txn") 
        
           }

in pkg/sql.(*planner).CheckPrivilegeForUser

cockroach/pkg/sql/authorization.go

Lines 742 to 744 in 5b78463

    
           case catalog.SchemaUserDefined: 
        
           	return p.CheckPrivilegeForUser(ctx, scDesc, privilege.USAGE, p.User()) 
        
           default:

in pkg/sql.(*planner).canResolveDescUnderSchema

cockroach/pkg/sql/opt_catalog.go

Lines 213 to 215 in 5b78463

    
           // Ensure that the current user can access the target schema. 
        
           if err := oc.planner.canResolveDescUnderSchema(ctx, prefix.Schema, desc); err != nil { 
        
           	return nil, cat.DataSourceName{}, err

in pkg/sql.(*optCatalog).ResolveDataSource

cockroach/pkg/sql/opt/optbuilder/util.go

Lines 605 to 607 in 5b78463

    
           } 
        
           ds, resName, err := b.catalog.ResolveDataSource(b.ctx, flags, tn) 
        
           if err != nil {

in pkg/sql/opt/optbuilder.(*Builder).resolveDataSource

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 103 to 105 in 5b78463


	ds, depName, resName := b.resolveDataSource(tn, privilege.SELECT)

in pkg/sql/opt/optbuilder.(*Builder).buildDataSource

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 59 to 61 in 5b78463


	outScope = b.buildDataSource(source.Expr, indexFlags, locking, inScope)

in pkg/sql/opt/optbuilder.(*Builder).buildDataSource

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 1154 to 1156 in 5b78463

    
           ) (outScope *scope) { 
        
           	outScope = b.buildDataSource(tables[0], nil /* indexFlags */, locking, inScope)

in pkg/sql/opt/optbuilder.(*Builder).buildFromTablesRightDeep

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 1131 to 1133 in 5b78463

    
           	} 
        
           	return b.buildFromTablesRightDeep(tables, locking, inScope) 
        
           }

in pkg/sql/opt/optbuilder.(*Builder).buildFromTables

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 1058 to 1060 in 5b78463

    
           if len(from.Tables) > 0 { 
        
           	outScope = b.buildFromTables(from.Tables, locking, inScope) 
        
           } else {

in pkg/sql/opt/optbuilder.(*Builder).buildFrom

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 979 to 981 in 5b78463

    
           ) (outScope *scope) { 
        
           	fromScope := b.buildFrom(sel.From, locking, inScope)

in pkg/sql/opt/optbuilder.(*Builder).buildSelectClause

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 927 to 929 in 5b78463

    
           case *tree.SelectClause: 
        
           	outScope = b.buildSelectClause(t, orderBy, locking, desiredTypes, inScope)

in pkg/sql/opt/optbuilder.(*Builder).buildSelectStmtWithoutParens

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 900 to 902 in 5b78463

    
           return b.processWiths(with, inScope, func(inScope *scope) *scope { 
        
           	return b.buildSelectStmtWithoutParens( 
        
           		wrapped, orderBy, limit, locking, desiredTypes, inScope,

in pkg/sql/opt/optbuilder.(*Builder).buildSelect.func1

cockroach/pkg/sql/opt/optbuilder/with.go

Lines 114 to 116 in 5b78463

    
           inScope.atRoot = false 
        
           outScope := buildStmt(inScope) 
        
           outScope.expr = b.buildWiths(outScope.expr, correlatedCTEs)

in pkg/sql/opt/optbuilder.(*Builder).processWiths

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 899 to 901 in 5b78463

    
           return b.processWiths(with, inScope, func(inScope *scope) *scope { 
        
           	return b.buildSelectStmtWithoutParens(

in pkg/sql/opt/optbuilder.(*Builder).buildSelect

cockroach/pkg/sql/opt/optbuilder/builder.go

Lines 280 to 282 in 5b78463

    
           case *tree.Select: 
        
           	return b.buildSelect(stmt, noRowLocking, desiredTypes, inScope)

in pkg/sql/opt/optbuilder.(*Builder).buildStmt

cockroach/pkg/sql/opt/optbuilder/with.go

Lines 193 to 195 in 5b78463

    
           if !isRecursive { 
        
           	cteScope := b.buildStmt(cte.Stmt, nil /* desiredTypes */, inScope) 
        
           	cteScope.removeHiddenCols()

in pkg/sql/opt/optbuilder.(*Builder).buildCTE

cockroach/pkg/sql/opt/optbuilder/with.go

Lines 138 to 140 in 5b78463

    
           hasRecursive = hasRecursive || with.Recursive 
        
           cteExpr, cteCols, cteOrdering := b.buildCTE(cte, outScope, with.Recursive)

in pkg/sql/opt/optbuilder.(*Builder).buildCTEs

cockroach/pkg/sql/opt/optbuilder/with.go

Lines 111 to 113 in 5b78463

    
           var correlatedCTEs cteSources 
        
           inScope, correlatedCTEs = b.buildCTEs(with, inScope) 
        
           prevAtRoot := inScope.atRoot

in pkg/sql/opt/optbuilder.(*Builder).processWiths

cockroach/pkg/sql/opt/optbuilder/select.go

Lines 899 to 901 in 5b78463

    
           return b.processWiths(with, inScope, func(inScope *scope) *scope { 
        
           	return b.buildSelectStmtWithoutParens(

in pkg/sql/opt/optbuilder.(*Builder).buildSelect

cockroach/pkg/sql/opt/optbuilder/builder.go

Lines 280 to 282 in 5b78463

    
           case *tree.Select: 
        
           	return b.buildSelect(stmt, noRowLocking, desiredTypes, inScope)

in pkg/sql/opt/optbuilder.(*Builder).buildStmt

cockroach/pkg/sql/opt/optbuilder/builder.go

Lines 240 to 242 in 5b78463

    
           b.ctes = nil 
        
           outScope = b.buildStmt(stmt, desiredTypes, inScope) 
        
           // Build With operators for any CTEs hoisted to the top level.

in pkg/sql/opt/optbuilder.(*Builder).buildStmtAtRoot

cockroach/pkg/sql/opt/optbuilder/builder.go

Lines 214 to 216 in 5b78463

    
           // and physical properties. 
        
           outScope := b.buildStmtAtRoot(b.stmt, nil /* desiredTypes */)

in pkg/sql/opt/optbuilder.(*Builder).Build

cockroach/pkg/sql/plan_opt.go

Lines 398 to 400 in 5b78463

    
           bld.KeepPlaceholders = true 
        
           if err := bld.Build(); err != nil { 
        
           	return nil, err

in pkg/sql.(*optPlanningCtx).buildReusableMemo

cockroach/pkg/sql/plan_opt.go

Lines 141 to 143 in 5b78463

    
           memo, err := opc.buildReusableMemo(ctx) 
        
           if err != nil {

in pkg/sql.(*planner).prepareUsingOptimizer

cockroach/pkg/sql/conn_executor_prepare.go

Lines 277 to 279 in 5b78463

    
           // and execute paths. 
        
           flags, err := p.prepareUsingOptimizer(ctx) 
        
           if err != nil {

in pkg/sql.(*connExecutor).populatePrepared

cockroach/pkg/sql/conn_executor_prepare.go

Lines 234 to 236 in 5b78463

    
           p.semaCtx.Annotations = tree.MakeAnnotations(stmt.NumAnnotations) 
        
           flags, err = ex.populatePrepared(ctx, txn, placeholderHints, p) 
        
           return err

in pkg/sql.(*connExecutor).prepare.func1

cockroach/pkg/sql/conn_executor_prepare.go

Lines 239 to 241 in 5b78463

    
           // Use the existing transaction. 
        
           if err := prepare(ctx, ex.state.mu.txn); err != nil && origin != PreparedStatementOriginSessionMigration { 
        
           	return nil, err

in pkg/sql.(*connExecutor).prepare

cockroach/pkg/sql/conn_executor_prepare.go

Lines 101 to 103 in 5b78463

    
           // Prepare the query. This completes the typing of placeholders. 
        
           prepared, err := ex.prepare(ctx, stmt, placeholderHints, rawTypeHints, origin) 
        
           if err != nil {

in pkg/sql.(*connExecutor).addPreparedStmt

cockroach/pkg/sql/conn_executor_prepare.go

Lines 62 to 64 in 5b78463

    
           stmt := makeStatement(parseCmd.Statement, ex.generateID()) 
        
           _, err := ex.addPreparedStmt( 
        
           	ctx,

in pkg/sql.(*connExecutor).execPrepare

cockroach/pkg/sql/conn_executor.go

Lines 1974 to 1976 in 5b78463

    
           	stmtCtx := withStatement(ctx, ex.curStmtAST) 
        
           	ev, payload = ex.execPrepare(stmtCtx, tcmd) 
        
           case DescribeStmt:

in pkg/sql.(*connExecutor).execCmd

cockroach/pkg/sql/conn_executor.go

Lines 1798 to 1800 in 5b78463

    
           var err error 
        
           if err = ex.execCmd(); err != nil { 
        
           	if errors.IsAny(err, io.EOF, errDrainingComplete) {

in pkg/sql.(*connExecutor).run

cockroach/pkg/sql/conn_executor.go

Lines 746 to 748 in 5b78463

    
           	}() 
        
           	return h.ex.run(ctx, s.pool, reserved, cancel) 
        
           }

in pkg/sql.(*Server).ServeConn

pkg/sql/authorization.go in pkg/sql.(*planner).CheckPrivilegeForUser at line 129
pkg/sql/authorization.go in pkg/sql.(*planner).canResolveDescUnderSchema at line 743
pkg/sql/opt_catalog.go in pkg/sql.(*optCatalog).ResolveDataSource at line 214
pkg/sql/opt/optbuilder/util.go in pkg/sql/opt/optbuilder.(*Builder).resolveDataSource at line 606
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildDataSource at line 104
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildDataSource at line 60
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildFromTablesRightDeep at line 1155
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildFromTables at line 1132
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildFrom at line 1059
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildSelectClause at line 980
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildSelectStmtWithoutParens at line 928
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildSelect.func1 at line 901
pkg/sql/opt/optbuilder/with.go in pkg/sql/opt/optbuilder.(*Builder).processWiths at line 115
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildSelect at line 900
pkg/sql/opt/optbuilder/builder.go in pkg/sql/opt/optbuilder.(*Builder).buildStmt at line 281
pkg/sql/opt/optbuilder/with.go in pkg/sql/opt/optbuilder.(*Builder).buildCTE at line 194
pkg/sql/opt/optbuilder/with.go in pkg/sql/opt/optbuilder.(*Builder).buildCTEs at line 139
pkg/sql/opt/optbuilder/with.go in pkg/sql/opt/optbuilder.(*Builder).processWiths at line 112
pkg/sql/opt/optbuilder/select.go in pkg/sql/opt/optbuilder.(*Builder).buildSelect at line 900
pkg/sql/opt/optbuilder/builder.go in pkg/sql/opt/optbuilder.(*Builder).buildStmt at line 281
pkg/sql/opt/optbuilder/builder.go in pkg/sql/opt/optbuilder.(*Builder).buildStmtAtRoot at line 241
pkg/sql/opt/optbuilder/builder.go in pkg/sql/opt/optbuilder.(*Builder).Build at line 215
pkg/sql/plan_opt.go in pkg/sql.(*optPlanningCtx).buildReusableMemo at line 399
pkg/sql/plan_opt.go in pkg/sql.(*planner).prepareUsingOptimizer at line 142
pkg/sql/conn_executor_prepare.go in pkg/sql.(*connExecutor).populatePrepared at line 278
pkg/sql/conn_executor_prepare.go in pkg/sql.(*connExecutor).prepare.func1 at line 235
pkg/sql/conn_executor_prepare.go in pkg/sql.(*connExecutor).prepare at line 240
pkg/sql/conn_executor_prepare.go in pkg/sql.(*connExecutor).addPreparedStmt at line 102
pkg/sql/conn_executor_prepare.go in pkg/sql.(*connExecutor).execPrepare at line 63
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).execCmd at line 1975
pkg/sql/conn_executor.go in pkg/sql.(*connExecutor).run at line 1799
pkg/sql/conn_executor.go in pkg/sql.(*Server).ServeConn at line 747

Tag	Value
Cockroach Release	`v22.1.0`
Cockroach SHA:	`5b78463`
Platform	linux amd64
Distribution	CCL
Environment	v22.1.0
Command	server
Go Version	``
# of CPUs
# of Goroutines

Jira issue: CRDB-16744

The text was updated successfully, but these errors were encountered:

rafiss · 2022-06-28T21:01:52Z

dupe of #82034

cockroach-teamcity added C-bug Code not up to spec/doc, specs & docs deemed correct. Solution expected to change code/behavior. O-sentry Originated from an in-the-wild panic report. labels Jun 15, 2022

rafiss closed this as completed Jun 28, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

sentry: authorization.go:129: cannot use CheckPrivilege without a txn (1) assertion failure Wraps: (2) attached stack trace -- stack trace: | github.com/cockroachdb/cockroach/pkg/sql.(*planner).CheckPrivi... #82942

sentry: authorization.go:129: cannot use CheckPrivilege without a txn (1) assertion failure Wraps: (2) attached stack trace -- stack trace: | github.com/cockroachdb/cockroach/pkg/sql.(*planner).CheckPrivi... #82942

cockroach-teamcity commented Jun 15, 2022 •

edited by cockroach-jira-scripts

Loading

rafiss commented Jun 28, 2022

sentry: authorization.go:129: cannot use CheckPrivilege without a txn (1) assertion failure Wraps: (2) attached stack trace -- stack trace: | github.com/cockroachdb/cockroach/pkg/sql.(*planner).CheckPrivi... #82942

sentry: authorization.go:129: cannot use CheckPrivilege without a txn (1) assertion failure Wraps: (2) attached stack trace -- stack trace: | github.com/cockroachdb/cockroach/pkg/sql.(*planner).CheckPrivi... #82942

Comments

cockroach-teamcity commented Jun 15, 2022 • edited by cockroach-jira-scripts Loading

rafiss commented Jun 28, 2022

cockroach-teamcity commented Jun 15, 2022 •

edited by cockroach-jira-scripts

Loading