NotificationsYou must be signed in to change notification settings
Fork49
Star707

Commit0c5bb5c

Alexander Korotkov

committed

Fix some clang analyzer complaints.

1 parent6f1efc3 commit0c5bb5cCopy full SHA for 0c5bb5c

File tree

2 files changed

+10

-0

lines changed

2 files changed

+10

-0

lines changed

`‎jsonb_gin_ops.c‎`

Lines changed: 9 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -691,6 +691,8 @@ gin_extract_jsonb_value_path_internal(Jsonb jb, int32 nentries, uint32 **bloom`
`691`	`691`	`stack->parent=tmp;`
`692`	`692`	`break;`
`693`	`693`	`caseWJB_KEY:`
	`694`	`+if (!stack)/* should never happen */`
	`695`	`+elog(ERROR,"error jsonb iteration");`
`694`	`696`	`stack->hash=0;`
`695`	`697`	`JsonbHashScalarValue(&v,&stack->hash);`
`696`	`698`	`break;`
`@@ -709,6 +711,8 @@ gin_extract_jsonb_value_path_internal(Jsonb jb, int32 nentries, uint32 **bloom`
`709`	`711`	`break;`
`710`	`712`	`caseWJB_END_OBJECT:`
`711`	`713`	`/* Pop the stack */`
	`714`	`+if (!stack)/* should never happen */`
	`715`	`+elog(ERROR,"error jsonb iteration");`
`712`	`716`	`tmp=stack->parent;`
`713`	`717`	`pfree(stack);`
`714`	`718`	`stack=tmp;`
`@@ -1109,6 +1113,9 @@ gin_extract_jsonb_path_value_internal(Jsonb jb, int32 nentries)`
`1109`	`1113`	`entries= (Datum)repalloc(entries,sizeof(Datum)total);`
`1110`	`1114`	`}`
`1111`	`1115`
	`1116`	`+if (!stack)/* should never happen */`
	`1117`	`+elog(ERROR,"error jsonb iteration");`
	`1118`	`+`
`1112`	`1119`	`switch (r)`
`1113`	`1120`	`{`
`1114`	`1121`	`caseWJB_BEGIN_ARRAY:`
`@@ -1131,6 +1138,8 @@ gin_extract_jsonb_path_value_internal(Jsonb jb, int32 nentries)`
`1131`	`1138`	`break;`
`1132`	`1139`	`caseWJB_KEY:`
`1133`	`1140`	`/* Initialize hash from parent */`
	`1141`	`+if (!stack->parent)/* should never happen */`
	`1142`	`+elog(ERROR,"error jsonb iteration");`
`1134`	`1143`	`stack->hash=stack->parent->hash;`
`1135`	`1144`	`JsonbHashScalarValue(&v,&stack->hash);`
`1136`	`1145`	`break;`

`‎jsquery_extract.c‎`

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,7 @@ recursiveExtract(JsQueryItem jsq, bool not, bool indirect, PathItem path)`
`48`	`48`	`ExtractedNodeTypetype;`
`49`	`49`	`JsQueryItemelem,e;`
`50`	`50`
	`51`	`+e.hint= false;`
`51`	`52`	`check_stack_depth();`
`52`	`53`
`53`	`54`	`switch(jsq->type)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

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

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit0c5bb5c

File tree

2 files changed

2 files changed

`‎jsonb_gin_ops.c‎`

`‎jsquery_extract.c‎`

0 commit comments