[PATCH v2 02/21] clone: use xfopen() instead of fopen()

From: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
Date: May 3, 2017, 10:16 UTC
Message-ID: <20170503101706.9223-3-pclouds@gmail.com>
In-Reply-To: <20170503101706.9223-1-pclouds@gmail.com>

This code uses the result FILE pointer right away without the NULL check. Let's use xfopen() and die if we could not open the file. That's still better than crashing,

Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@gmail.com>
---
 builtin/clone.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/clone.c b/builtin/clone.c
index de85b85254..dde4fe73af 100644
--- a/builtin/clone.c
+++ b/builtin/clone.c
@@ -357,7 +357,7 @@ static void copy_alternates(struct strbuf *src, struct strbuf *dst,
 	 * to turn entries with paths relative to the original
 	 * absolute, so that they can be used in the new repository.
 	 */
-	FILE *in = fopen(src->buf, "r");
+	FILE *in = xfopen(src->buf, "r");
 	struct strbuf line = STRBUF_INIT;
 
 	while (strbuf_getline(&line, in) != EOF) {

-- 
2.11.0.157.gd943d85