[devel] I: xorg-server, dbus: SELinux support breaks software

Alexey Gladkov legion на altlinux.ru
Пн Авг 30 10:30:20 UTC 2010


30.08.2010 14:21, Dmitry V. Levin wrote:
> В коде is_selinux_enabled() нет проверки на невозможность прочитать.

  38         /* Drop back to detecting it the long way. */
  39         fp = fopen("/proc/filesystems", "r");
  40         if (!fp)
  41                 return -1;

Здесь мы возвращаем -1. Зачем тут -1, почему не goto out; ?

  44         while ((num = getline(&buf, &len, fp)) != -1) {
  45                 if (strstr(buf, "selinuxfs")) {
  46                         enabled = 1;
  47                         break;
  48                 }
  49         }
  50
  51         if (num < 0)
  52                 goto out;

Дим, проверка есть и в этом случае вернётся 0.

> Тем не менее, эта функция определена таким образом, что может давать
> три вида ответа: "да", "нет" и "нет способа определить".

-- 
Rgrds, legion



Подробная информация о списке рассылки Devel