oxen-io / session-android

A private messenger for Android.
https://getsession.org
GNU General Public License v3.0
1.86k stars 170 forks source link

SES1251 - App crash on non alphanumeric RE-FIX #1445

Closed AL-Session closed 5 months ago

AL-Session commented 6 months ago

Contributor checklist


Description

Searching messages and entering a non-alphanumeric character as the first character caused an crash & restart. This was due to the search being performed after each character entered, and any illegal character entered as the 1st character is replaced with a space - so in effect we are searching on the string " " which was causing the error.

Guards have been put in place to prevent search queries on strings of " " which eliminates the error.

Tested by entering a variety of non-alphanumeric characters as first search letter of message search both in regular user-to-user comms & in a group- all fine now.