Как мирный reverse engineering помог чуть-чуть улучшить приложение Яндекс.Деньги

Награду то пообещали?)

А кто да? Г...? ;)

Думаю этот код не был отброшен из-за (пишу псевдокодом, так короче и понятнее, кому интересно — сам посмотрит как это выглядит в Smali):

public class CoreApplication implements LocationListener {
 
  // ...неинтересный код был тут

  public static CoreApplication getCoreApplication() {
    if (coreApplication == null)
      throw new RuntimeException("here CoreApplication must exists!");
    return coreApplication;
  }
  
  private static void onNativeCrashed() {
    Class localClass = getCoreApplication().params.getCrashHandlerClass();
    if ((localClass != null) && (!getAppBuildIdFromNative().contains("master_market"))) {
      new RuntimeException("crashed here (native trace should follow after the Java trace)");
      new StringBuilder("App name is ").append(getAppNameFromNative());
      new StringBuilder("Build ID is ").append(getAppBuildIdFromNative());
      if (getAppBuildIdFromNative().length() > 32)
        applicationContext.startActivity(new Intent(applicationContext, localClass).setFlags(268435456));
    }
	
	// ...неинтересный код был тут
}

Обратите внимание на метод onNativeCrashed(). Он вызывается при краше native библиотеки которая работает с Яндекс.Картами. Метод получает класс, унаследованный от CrashHandler (в котором и как раз и живет тот самый безобразный метод sendBug(String paramString)) путём вызова

Class localClass = getCoreApplication().params.getCrashHandlerClass();

Если класс получен успешно (см. дальше условия if ((localClass != null)...) и если native-библиотека которая отвечает за взаимодействие с Яндекс.Картами вернула правильный пароль :) — стартует activity, из которой потом можно будет вызвать этот самый гадкий sendBug(String paramString).

Но есть одно «но»! Если посмотреть на метод getCrashHandlerClass(), то мы увидим что он всегда возвращает null:

public class MapsCoreApplicationParams extends CoreApplicationParams
{
  // ...ля-ля-ля, не важно
  
  public Class getCrashHandlerClass()
  {
    return null;
  }
  
  // ...ля-ля-ля, тоже не важно
}

Поэтому класс никогда не будет получен, а следовательно activity не стартует и метод sendBug(String paramString) никогда не будет вызван. Как видите, тут все запутано как бразильском сериале :) Думаю поэтому компилятор не смог до конца вкурить все эти связи и стремный код на всякий случай оставил.

Я не стал включать все эти исследования в статью — решил ограничится общей фразой про то что мол тщательное исследование кода приложения заставляет думать что ля-ля-ля…

Возможную причину я написал выше, но может Вы и правы — дело действительно в том, что Java не склонна выкидывать public и protected методы. Честно говоря точно не знаю.

данные перед отправкой в штатный механизм можно и зашифровать, не должно быть проблемой.

Извиняюсь, проглядел абзац, мне показалось, что вы работаете в Яндексе. Троллинг не получился.

Дима, напишите мне в личку или по job.rabota@gmail.com есть вопрос, а то не могу найти контактных данных в профиле. Спасибо