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

    +139

    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
    bool aiccu_os_install(void)
{
    /* Check if IPv6 support is available */
    if (access("/proc/net/if_inet6", F_OK))
    {
        /* Doing the modprobe doesn't guarantee success unfortunately */
        (void)system("modprobe -q ipv6 2>/dev/null >/dev/null");

        /* Thus test it again */
        if (access("/proc/net/if_inet6", F_OK))
        {
            dolog(LOG_ERR, "No IPv6 Stack found! Please check your kernel and module configuration\n");
            return false;
        }
    }

    /* Try to load modules (SIT tunnel, TUN/TAP)
     * They can be kernel builtins and there is no easy
     * way to check if they are loaded/built except for
     * trying to use them and fail at that point
     */
    (void)system("modprobe -q sit 2>/dev/null >/dev/null");
    (void)system("modprobe -q tun 2>/dev/null >/dev/null");

    return true;
}

    Запостил: raorn, 27 Июня 2011

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

    • ava lucidfox 28.06.2011 13:21 # 0

      А в чём хохма? Кроме (void) в скобках.
      Ответить

      • ava RaZeR 28.06.2011 13:30 # −3

        Из /dev/null в /dev/null, может быть? Лично я не знаю, зачем это тут и для чего сию конструкцию можно использовать...
        Ответить

        • ava macGovno 28.06.2011 13:43 # +3

          Это чтобы и stdout и stderr в null слить, правильнее писать &>/dev/null
          Ответить

          • ava guest 23.03.2012 15:56 # 0

            &>/dev/null это расширение bash и работает не везде (в дебиане, например использует dash для всего неинтеррактивного).
            Ответить

      • ava macGovno 28.06.2011 13:52 # 0

        Мне кажется должно быть что то вроде:
        bool aiccu_os_install(void)
{
	return !system("modprobe -q ipv6 &>/dev/null") && 
	  !system("modprobe -q sit &>/dev/null") && 
	  !system("modprobe -q tun &>/dev/null");
}


        Но могу ошибаться, не пробовал использовать.
        Ответить

        • ava kipar 28.06.2011 16:13 # 0

          /* Doing the modprobe doesn't guarantee success unfortunately */
          Ответить

          • ava macGovno 28.06.2011 17:19 # 0

            Sure, but doesn’t it return a non zero value in this case?
            Ответить

          • ava eth0 28.06.2011 19:40 # 0

            Разве lsmod не выведет загруженные модули?
            Ответить

            • ava rat4 28.06.2011 20:38 # 0

              Что-то вроде `lsmod | grep -E 'ipv6|sit|tun' | wc -l` да? :-)
              Ответить

              • ava eth0 28.06.2011 22:05 # 0

                Как вариант вполне покатит. Правда, не вспомню, потянет ли system конвейерное выполнение или нет.
                Ответить

                • ava guest6 9 дней назад # 0

                  конвеером шел занимается

                  modprobe может обламаться, если не сомжет найти нужные модули, представлябющие нужные модулю символы,он же по сути граф зависимостей строит
                  Ответить

                  • ava guest6 9 дней назад # 0

                    Глянул про system: Calls the host environment's command processor.

                    Так что всё норм, system позовёт шелл, он символы конвейера распарсит.

                    Вот всякие exec и spawn (не помню их сигнатур), которые напрямую вызывают бинарник, обломаются, конечно, им нужно явно ссылку на sh подставлять.
                    Ответить

                    • ava guest6 9 дней назад # 0

                      да, ты прав: `sh` должен быть
                      https://pubs.opengroup.org/onlinepubs/9799919799/functions/system.html

                      брать путь к ш реализаиця должна через confstr, но примерно во всех юниксах там почти всегда будет /bin/sh (но в теории может и не)


                      там даже более того написино

                      >>>
                      Finally, POSIX.1-2024 requires the command to be interpreted as in the shell command language defined in the Shell and Utilities volume of POSIX.1-2024.
                      >>>
                      Ответить

                      • ava guest6 9 дней назад # 0

                        Вот, там даже рекомендуют реализовать system так, чтобы она под капотом выполняла
                        execl(<shell path>, "sh", "-c", "--", command, (char *)0);

                        Либо, чтобы выполняла свой код, но результат был такой же, как после этого вызова.

                        Вот в чём отличие system от execl.
                        Ответить

    • ava guest8 08.04.2019 21:01 # −999

      показать все, что скрытоvanished
      Ответить

    • ava guest8 09.04.2019 11:05 # −999

      показать все, что скрытоvanished
      Ответить

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