Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Commit d6f7efa

Browse files
committed
Merge pull request #5802 from matrix-org/erikj/deny_redacting_different_room
2 parents 9d0cc25 + 58af30a commit d6f7efa

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

changelog.d/5802.misc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Deny redactions of events sent in a different room.

synapse/handlers/message.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -795,14 +795,16 @@ def is_inviter_member_event(e):
795795
get_prev_content=False,
796796
allow_rejected=False,
797797
allow_none=True,
798-
check_room_id=event.room_id,
799798
)
800799

801800
# we can make some additional checks now if we have the original event.
802801
if original_event:
803802
if original_event.type == EventTypes.Create:
804803
raise AuthError(403, "Redacting create events is not permitted")
805804

805+
if original_event.room_id != event.room_id:
806+
raise SynapseError(400, "Cannot redact event from a different room")
807+
806808
prev_state_ids = yield context.get_prev_state_ids(self.store)
807809
auth_events_ids = yield self.auth.compute_auth_events(
808810
event, prev_state_ids, for_verification=True

0 commit comments

Comments
 (0)