NotificationsYou must be signed in to change notification settings
Fork49
Star705

Commit580f6cb

Alexander Korotkov

committed

Fix some clang analyzer complaints.

1 parent0dc0ef4 commit580f6cbCopy full SHA for 580f6cb

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!

Commit580f6cb

File tree

2 files changed

2 files changed

`‎jsonb_gin_ops.c`

`‎jsquery_extract.c`

0 commit comments