empathicqubit/Signal-Android
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Do not attempt to retrieve a profile when it is not a number.

Browse source 
Fixes #8855
This commit is contained in:
Alan Evans 2019-06-13 18:20:25 -04:00 • committed by Greyson Parrelli
parent ae784db80d
commit 42a5504f0d
1 changed files with 11 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

26
src/org/thoughtcrime/securesms/jobs/RetrieveProfileJob.java
View file

@ -33,7 +33,6 @@ import org.whispersystems.signalservice.api.crypto.UnidentifiedAccessPair;
import org.whispersystems.signalservice.api.profiles.SignalServiceProfile;
import org.whispersystems.signalservice.api.push.SignalServiceAddress;
import org.whispersystems.signalservice.api.push.exceptions.NonSuccessfulResponseCodeException;
import org.whispersystems.signalservice.api.util.InvalidNumberException;
import java.io.IOException;
import java.util.List;
@ -50,7 +49,7 @@ public class RetrieveProfileJob extends BaseJob implements InjectableType {
@Inject SignalServiceMessageReceiver receiver;
private Recipient recipient;
private final Recipient recipient;
public RetrieveProfileJob(@NonNull Recipient recipient) {
this(new Job.Parameters.Builder()
@ -76,13 +75,9 @@ public class RetrieveProfileJob extends BaseJob implements InjectableType {
}
@Override
public void onRun() throws IOException, InvalidKeyException {
try {
if (recipient.isGroupRecipient()) handleGroupRecipient(recipient);
else handleIndividualRecipient(recipient);
} catch (InvalidNumberException e) {
Log.w(TAG, e);
}
public void onRun() throws IOException {
if (recipient.isGroupRecipient()) handleGroupRecipient(recipient);
else handleIndividualRecipient(recipient);
}
@Override
@ -93,9 +88,12 @@ public class RetrieveProfileJob extends BaseJob implements InjectableType {
@Override
public void onCanceled() {}
private void handleIndividualRecipient(Recipient recipient)
throws IOException, InvalidKeyException, InvalidNumberException
{
private void handleIndividualRecipient(Recipient recipient) throws IOException {
if (recipient.getAddress().isPhone()) handlePhoneNumberRecipient(recipient);
else Log.w(TAG, "Skipping fetching profile of non-phone recipient");
}
private void handlePhoneNumberRecipient(Recipient recipient) throws IOException {
String number = recipient.getAddress().toPhoneString();
Optional<UnidentifiedAccess> unidentifiedAccess = getUnidentifiedAccess(recipient);
@ -117,9 +115,7 @@ public class RetrieveProfileJob extends BaseJob implements InjectableType {
setUnidentifiedAccessMode(recipient, profile.getUnidentifiedAccess(), profile.isUnrestrictedUnidentifiedAccess());
}
private void handleGroupRecipient(Recipient group)
throws IOException, InvalidKeyException, InvalidNumberException
{
private void handleGroupRecipient(Recipient group) throws IOException {
List<Recipient> recipients = DatabaseFactory.getGroupDatabase(context).getGroupMembers(group.getAddress().toGroupString(), false);
for (Recipient recipient : recipients) {