1. Assembler / Говнокод #11531

    +138

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    .text:00018584                 EXPORT get_msg_sequence
    .text:00018584 get_msg_sequence                        ; CODE XREF: IPC_send_singleIPC+6p
    .text:00018584                                         ; __IPC_send_singleIPC+6p
    .text:00018584                 LDR             R0, =0x7800C
    .text:00018586                 PUSH            {R4,LR}
    .text:00018588                 ADD             R0, PC
    .text:0001858A                 LDR             R0, [R0]
    .text:0001858C                 LDRB            R3, [R0]
    .text:0001858E                 CBZ             R3, loc_185A2
    .text:00018590                 LDR             R1, =(aRil - 0x1859C)
    .text:00018592                 MOVS            R0, #6
    .text:00018594                 LDR             R2, =(aS_0 - 0x1859E)
    .text:00018596                 LDR             R3, =(aGet_msg_sequen - 0x185A0)
    .text:00018598                 ADD             R1, PC  ; "RIL"
    .text:0001859A                 ADD             R2, PC  ; "%s()"
    .text:0001859C                 ADD             R3, PC  ; "get_msg_sequence"
    .text:0001859E                 BLX             __android_log_print
    .text:000185A2
    .text:000185A2 loc_185A2                               ; CODE XREF: get_msg_sequence+Aj
    .text:000185A2                 LDR             R3, =(unk_98E7E - 0x185A8)
    .text:000185A4                 ADD             R3, PC
    .text:000185A6                 LDRB            R2, [R3]
    .text:000185A8                 CMP             R2, #0xFD
    .text:000185AA                 BLS             loc_185B0
    .text:000185AC                 MOVS            R2, #0
    .text:000185AE                 B               loc_185B2
    .text:000185B0 ; ---------------------------------------------------------------------------
    .text:000185B0
    .text:000185B0 loc_185B0                               ; CODE XREF: get_msg_sequence+26j
    .text:000185B0                 ADDS            R2, #1
    .text:000185B2
    .text:000185B2 loc_185B2                               ; CODE XREF: get_msg_sequence+2Aj
    .text:000185B2                 LDR             R1, =(unk_98E7E - 0x185BA)
    .text:000185B4                 STRB            R2, [R3]
    .text:000185B6                 ADD             R1, PC
    .text:000185B8                 LDRB            R0, [R1]
    .text:000185BA                 POP             {R4,PC}
    .text:000185BA ; End of function get_msg_sequence

    Потрясающе эффективный выхлоп GCC. Компилировалось, вероятно, все-таки с включенной оптимизацией.

    Например, последовательность LDR R1/ADD R1,PC/LDRB R0, [R1] в конце вообще лишняя: достаточно было использовать R0 вместо R2.

    Запостил: Grindars, 04 Августа 2012

    Комментарии (8) RSS

    • > достаточно было использовать R0 вместо R2
      Можно пример? Что-то до меня не доходит, как поюзать R0 вместо R2 в приведенных вами инструкциях, которые R2 не используют...
      Ответить
      • Примерно так:

        .text:000185A2 loc_185A2                               ; CODE XREF: get_msg_sequence+Aj
        .text:000185A2                 LDR             R3, =(unk_98E7E - 0x185A8)
        .text:000185A4                 ADD             R3, PC
        .text:000185A6                 LDRB            R0, [R3]
        .text:000185A8                 CMP             R0, #0xFD
        .text:000185AA                 BLS             loc_185B0
        .text:000185AC                 MOVS            R0, #0
        .text:000185AE                 B               loc_185B2
        .text:000185B0 ; ---------------------------------------------------------------------------
        .text:000185B0
        .text:000185B0 loc_185B0                               ; CODE XREF: get_msg_sequence+26j
        .text:000185B0                 ADDS            R0, #1
        .text:000185B2
        .text:000185B2 loc_185B2                               ; CODE XREF: get_msg_sequence+2Aj
        .text:000185B4                 STRB            R0, [R3]
        .text:000185BA                 POP             {R4,PC}
        .text:000185BA ; End of function get_msg_sequence


        R0 - возвращаемое значение по ABI, там сишный прототип int get_msg_sequence();
        Ответить
        • А unk_98E7E случаем не описана как volatile?
          Ответить
          • Возможно, я не знаю. Но смысла ее так описывать здесь нет все равно, и из других процедур она не используется.
            Ответить
            • > и из других процедур она не используется.
              А зачем ее тогда описали в глобале? Компиляторы, вообще говоря, очень плохо умеют оптимизировать все, что связано с глобальными переменными...

              P.S. В общем, мне кажется, что тут не только выхлоп GCC попахивает, но и исходный код, который мы, к сожалению, не увидим...
              Ответить
              • Она не обязательно глобальная. Код, по сути, такой:
                int get_msg_sequence() {
                    if(debug_enabled)
                        LOGD("%s()", "get_msg_sequence");
                
                    static unsigned char sequence_counter;
                
                    if(sequence_counter < 0xFE) {
                        return ++sequence_counter;
                    } else {
                        sequence_counter = 0;
                    }
                }
                Ответить
                • Эх, потерял return для ветки else.
                  Ответить
                • Сорри, ступил я, это действительно мог быть статик.

                  А код, наверное был таким:
                  int get_msg_sequence() {
                      if(debug_enabled)
                          LOGD("%s()", "get_msg_sequence");
                  
                      static unsigned char sequence_counter;
                  
                      if(sequence_counter < 0xFE) {
                          ++sequence_counter;
                      } else {
                          sequence_counter = 0;
                      }
                      return sequence_counter;
                  }
                  Ответить

    Добавить комментарий