The file should be closed before return.

Signed-off-by: Cong Ding <ding...@gmail.com>
---
 tools/kvm/builtin-setup.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/kvm/builtin-setup.c b/tools/kvm/builtin-setup.c
index c5b0566..8b45c56 100644
--- a/tools/kvm/builtin-setup.c
+++ b/tools/kvm/builtin-setup.c
@@ -159,12 +159,12 @@ static int copy_passwd(const char *guestfs_name)
                return -1;
 
        ret = fprintf(file, "root:x:0:0:root:/root:/bin/sh\n");
-       if (ret < 0)
-               return ret;
+       if (ret > 0)
+               ret = 0;
 
        fclose(file);
 
-       return 0;
+       return ret;
 }
 
 static int make_guestfs_symlink(const char *guestfs_name, const char *path)
-- 
1.7.10.4

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

Reply via email to