<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 07/13/2014 10:35 PM, Colin Walters
wrote:<br>
</div>
<blockquote
cite="mid:1405283733.22551.141180761.62D52A97@webmail.messagingengine.com"
type="cite">
<pre wrap="">On Sat, Jul 12, 2014, at 06:48 AM, Zbigniew Jędrzejewski-Szmek wrote:
</pre>
<blockquote type="cite">
<pre wrap="">
please excuse my possibly ignorant questions, selinux is not my forte.
If the files are nonexistent, will this fail? But sysusers should be
able to create /etc from scratch.
</pre>
</blockquote>
<pre wrap="">
True, fixed.
</pre>
<blockquote type="cite">
<pre wrap="">Why cannot the same code as in write_string_file_atomic_label be used
instead?
</pre>
</blockquote>
<pre wrap="">
We're writing more than one line, and in addition we need to look up the
label for /etc/passwd even if we're operating on a chroot.
</pre>
<blockquote type="cite">
<pre wrap="">BTW, you're returning -1 here, but should be -errno.
</pre>
</blockquote>
<pre wrap="">
This code is gone now; new patch attached.
</pre>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
systemd-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:systemd-devel@lists.freedesktop.org">systemd-devel@lists.freedesktop.org</a>
<a class="moz-txt-link-freetext" href="http://lists.freedesktop.org/mailman/listinfo/systemd-devel">http://lists.freedesktop.org/mailman/listinfo/systemd-devel</a>
</pre>
</blockquote>
Colin,<br>
how does look<br>
<br>
# ls -Z /etc/passwd* /etc/group*<br>
<br>
with this patch?<br>
</body>
</html>