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).

        Cyrill
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to