Add NULL check before calling FValid on IMDId

gpAux/releng/releng.mk

@@ -120,7 +120,7 @@ sync_tools: opt_write_test /opt/releng/apache-ant
 	-Divyrepo.user=$(IVYREPO_USER) -Divyrepo.passwd="$(IVYREPO_PASSWD)" resolve);
 	@echo "Resolve finished";
 
-	LD_LIBRARY_PATH='' wget -O - https://github.com/greenplum-db/gporca/releases/download/v2.39.0/bin_orca_centos5_release.tar.gz | tar zxf - -C $(BLD_TOP)/ext/$(BLD_ARCH)
+	LD_LIBRARY_PATH='' wget -O - https://github.com/greenplum-db/gporca/releases/download/v2.39.2/bin_orca_centos5_release.tar.gz | tar zxf - -C $(BLD_TOP)/ext/$(BLD_ARCH)
 
 clean_tools: opt_write_test
 	@cd releng/make/dependencies; \

src/backend/gpopt/translate/CTranslatorDXLToScalar.cpp

@@ -1094,7 +1094,7 @@ CTranslatorDXLToScalar::PrelabeltypeFromDXLNodeScCast
 
 	Expr *pexprChild = PexprFromDXLNodeScalar(pdxlnChild, pmapcidvar);
 
-	if (pdxlop->PmdidFunc()->FValid())
+	if (IMDId::FValid(pdxlop->PmdidFunc()))
 	{
 		FuncExpr *pfuncexpr = MakeNode(FuncExpr);
 		pfuncexpr->funcid = CMDIdGPDB::PmdidConvert(pdxlop->PmdidFunc())->OidObjectId();

src/backend/gpopt/translate/CTranslatorUtils.cpp

@@ -289,7 +289,7 @@ CTranslatorUtils::Pdxltvf
 		// function returns record - use col names and types from query
 		pdrgdxlcd = PdrgdxlcdRecord(pmp, pidgtor, prte->eref->colnames, prte->funccoltypes);
 	}
-	else if (pmdType->FComposite() && pmdType->PmdidBaseRelation()->FValid())
+	else if (pmdType->FComposite() && IMDId::FValid(pmdType->PmdidBaseRelation()))
 	{
 		// function returns a "table" type or a user defined type
 		pdrgdxlcd = PdrgdxlcdComposite(pmp, pmda, pidgtor, pmdType);