On 08/30/2012 09:50 AM, Cyrill Gorcunov wrote:
> On Thu, Aug 30, 2012 at 09:36:37AM +0200, Sasha Levin wrote:
>> + char *new_env[] = { "TERM=linux", "DISPLAY=192.168.33.1:0",
>> + "HOME=/virt/home", NULL };
>> +
>> + mkdir("/virt/home", 0755);
>
> Please add check for mkdir error code. Frankly, this is a bad habbit
> to assume that mkdir never fails (this could be done on top of this
> series I think but should not be leaved without attention).
This is actually supposed to fail most of the times, since it runs as part of
the init :)
There's not much we can do if it fails though, and even at that point - this
failure isn't serious enough to even justify any further action on our end.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
