On Tue, Jul 30, 2013 at 1:52 PM, Zbigniew Jędrzejewski-Szmek
<zbys...@in.waw.pl> wrote:
> On Tue, Jul 30, 2013 at 11:50:54AM +0200, Christian Hesse wrote:
>> Hello everybody,
>>
>> for me this test fails and I think it is correct to skip it in case
>> permission is denied. Patch attached.
> It *is* an error in the installation or environment if
> unit files cannot be read. In almost all circumstances,
> reducing permissions on unit files doesn't introduce any
> extra security because later on they are all accessible
> from PID 1 over dbus. I agree that failing an assertion
> is not nice from the view of the user, what about
> exit(EXIT_FAILURE)?

Hmm, we assert() most of the test failures to see where things went wrong.

Would exit() make much of a difference in that case? Is it worth to
distinguish different types of failures in the test?

Kay
_______________________________________________
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

Reply via email to