Skip to content

Commit a15e12c

Browse files
committed
Remove any NULL characters from displaynames to avoid breaking database txn
1 parent 8f4dde5 commit a15e12c

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

synapse/handlers/user_directory.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -464,6 +464,10 @@ async def _handle_possible_remote_profile_change(
464464

465465
prev_name = prev_event.content.get("displayname")
466466
new_name = event.content.get("displayname")
467+
468+
# Replace any NULL characters in the name as these cannot be stored in the database
469+
new_name = new_name.replace("\x00", "\uFFFD")
470+
467471
# If the new name is an unexpected form, do not update the directory.
468472
if not isinstance(new_name, str):
469473
new_name = prev_name

0 commit comments

Comments
 (0)