Fix incorrect print format in json.c

michaelpq · michaelpq · commit f2130e77da51 · 2020-07-29T14:44:32.000+09:00
Oid is unsigned, so %u needs to be used and not %d. The code path involved here is not normally reachable, so no backpatch is done. Author: Justin Pryzby Discussion: https://postgr.es/m/20200728015523.GA27308@telsasoft.com
diff --git a/src/backend/utils/adt/json.c b/src/backend/utils/adt/json.c
@@ -458,7 +458,7 @@ JsonEncodeDateTime(char *buf, Datum value, Oid typid, const int *tzp)
 			}
 			break;
 		default:
-			elog(ERROR, "unknown jsonb value datetime type oid %d", typid);
+			elog(ERROR, "unknown jsonb value datetime type oid %u", typid);
 			return NULL;
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -458,7 +458,7 @@ JsonEncodeDateTime(char buf, Datum value, Oid typid, const int tzp)`
`458`	`458`	`}`
`459`	`459`	`break;`
`460`	`460`	`default:`
`461`		`- elog(ERROR, "unknown jsonb value datetime type oid %d", typid);`
	`461`	`+ elog(ERROR, "unknown jsonb value datetime type oid %u", typid);`
`462`	`462`	`return NULL;`
`463`	`463`	`}`
`464`	`464`