From 45a8e18d32f9bf4451e78f3e799ffcabbceccc59 Mon Sep 17 00:00:00 2001
From: Jose Leite <122051639+joseleite550@users.noreply.github.com>
Date: Fri, 24 Mar 2023 10:20:21 -0300
Subject: [PATCH] IDEMPIERE-5640 - Fix Cross Tenant Sending BroadCast Message
 (#1750)

* IDEMPIERE-5640 - Fix Cross Tenant Sending BroadCast Message

* IDEMPIERE-5640 - Add validation in fix Cross Tenant Sending BroadCast Message
---
 .../adempiere/model/MBroadcastMessage.java    | 21 ++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/org.adempiere.base/src/org/adempiere/model/MBroadcastMessage.java b/org.adempiere.base/src/org/adempiere/model/MBroadcastMessage.java
index f8077754f3..62d01cd954 100644
--- a/org.adempiere.base/src/org/adempiere/model/MBroadcastMessage.java
+++ b/org.adempiere.base/src/org/adempiere/model/MBroadcastMessage.java
@@ -104,14 +104,21 @@ public class MBroadcastMessage extends X_AD_BroadcastMessage implements Immutabl
 		MBroadcastMessage retValue = s_cache.get(ctx, key, e -> new MBroadcastMessage(ctx, e));
 		if (retValue == null)
 		{
-			retValue = new MBroadcastMessage (ctx, AD_BroadcastMessage_ID, (String)null);
-			if (retValue.get_ID() == AD_BroadcastMessage_ID)
-			{
-				s_cache.put(key, retValue, e -> new MBroadcastMessage(Env.getCtx(), e));
-				return retValue;
-			}
-			return null;
+			try {
+	            PO.setCrossTenantSafe();
+	            // Here we can receive messages from another tenant and discard them
+				retValue = new MBroadcastMessage (ctx, AD_BroadcastMessage_ID, (String)null);
+	        } finally {
+	            PO.clearCrossTenantSafe();
+	        }
+			if (retValue.get_ID() != AD_BroadcastMessage_ID)
+				return null;
+			s_cache.put(key, retValue, e -> new MBroadcastMessage(Env.getCtx(), e));
 		}
+		if (retValue != null
+			&& retValue.getAD_Client_ID() != 0
+			&& retValue.getAD_Client_ID() != Env.getAD_Client_ID(ctx))
+			return null;
 		return retValue;
 	}	//	get