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

    • Только если "<=" вместо "<" надо. А так - суть не улавливается.
      Ответить
      • Поясняю. Допустим у нас тюнер ловит от 88 до 108 мегагерц.
        88 - минимальная частота, 108 - максимальная.
        Теперь внимательно смотрим на условие.
        меньше максимальной или больше минимальной => fail.
        Что-то тут не так, правда?
        ОНО НИКОГДА РАБОТАТЬ НЕ БУДЕТ!!111
        Ответить
        • А.. Пардон. Такое фиг увидишь.
          Ответить
        • А где-нибудь в конфигурации, наверное, перепутаны местами min и max...
          "не знаю, у меня работает."
          Ответить
    • А вообще неожиданно printk и return -EINVAL;
      Ответить

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