Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit01fc652

Browse files
committed
Fix variable lifespan in ExecInitCoerceToDomain().
This undoes a mistake in1ec7679: domainval and domainnull weremeant to live across loop iterations, but they were incorrectlymoved inside the loop. The effect was only to emit useless extraEEOP_MAKE_READONLY steps, so it's not a big deal; nonetheless,back-patch to v13 where the mistake was introduced.Ranier VilelaDiscussion:https://postgr.es/m/CAEudQAqXuhbkaAp-sGH6dR6Nsq7v28_0TPexHOm6FiDYqwQD-w@mail.gmail.com
1 parent7d9ec07 commit01fc652

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

‎src/backend/executor/execExpr.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3137,6 +3137,8 @@ ExecInitCoerceToDomain(ExprEvalStep *scratch, CoerceToDomain *ctest,
31373137
ExprState*state,Datum*resv,bool*resnull)
31383138
{
31393139
DomainConstraintRef*constraint_ref;
3140+
Datum*domainval=NULL;
3141+
bool*domainnull=NULL;
31403142
ListCell*l;
31413143

31423144
scratch->d.domaincheck.resulttype=ctest->resulttype;
@@ -3183,8 +3185,6 @@ ExecInitCoerceToDomain(ExprEvalStep *scratch, CoerceToDomain *ctest,
31833185
foreach(l,constraint_ref->constraints)
31843186
{
31853187
DomainConstraintState*con= (DomainConstraintState*)lfirst(l);
3186-
Datum*domainval=NULL;
3187-
bool*domainnull=NULL;
31883188
Datum*save_innermost_domainval;
31893189
bool*save_innermost_domainnull;
31903190

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp