1. Си / Говнокод #12276

    +133

    1. 1
    2. 2
    3. 3
    4. 4
    if ((frequency < config->frequency_max) || (frequency > config->frequency_min)) {
      printk(KERN_ERR "%s: Frequency beyond limits, frequency=%d\n", __func__, frequency);
      return -EINVAL;
    }

    Коллега отыскал где-то в недрах dvb подсистемы, в драйвере mopll'ки TDA6651.

    Запостил: Necromant, 12 Декабря 2012

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

    • ava krypt 12.12.2012 19:35 # −1

      Только если "<=" вместо "<" надо. А так - суть не улавливается.
      Ответить

      • ava Necromant 12.12.2012 21:15 # +7

        Поясняю. Допустим у нас тюнер ловит от 88 до 108 мегагерц.
        88 - минимальная частота, 108 - максимальная.
        Теперь внимательно смотрим на условие.
        меньше максимальной или больше минимальной => fail.
        Что-то тут не так, правда?
        ОНО НИКОГДА РАБОТАТЬ НЕ БУДЕТ!!111
        Ответить

        • ava krypt 12.12.2012 22:09 # 0

          А.. Пардон. Такое фиг увидишь.
          Ответить

        • ava Steve_Brown 13.12.2012 13:01 # 0

          А где-нибудь в конфигурации, наверное, перепутаны местами min и max...
          "не знаю, у меня работает."
          Ответить

    • ava absolut 12.12.2012 22:14 # 0

      А вообще неожиданно printk и return -EINVAL;
      Ответить

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