Merge pull request #2895 from tomponline/tp-2019-03-05/minor-fix

conf: Fixes uninitialised variable. > @brauner clang is failing but gcc passes, would it be OK to initialise this to 0? He, unfortunately not. :) What would happen now is that you'd close `stdin` on a standard system. What you'd want is `= -EBADF`. :)

Merge pull request #2895 from tomponline/tp-2019-03-05/minor-fix
b5bd9e11 · Christian Brauner · GitHub · 859ce011 · 912c4bb4 · b5bd9e11
Unverified Commit b5bd9e11 authored Mar 05, 2019 by Christian Brauner Committed by GitHub Mar 05, 2019
Show whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

conf.c src/lxc/conf.c +1 -1

No files found.
--- a/src/lxc/conf.c
+++ b/src/lxc/conf.c
@@ -2770,7 +2770,7 @@ struct lxc_conf *lxc_conf_init(void)
 int write_id_mapping(enum idtype idtype, pid_t pid, const char *buf,
 		     size_t buf_size)
 {
-	__do_close_prot_errno int fd;
+	__do_close_prot_errno int fd = 0;
 	int ret;
 	char path[PATH_MAX];