From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Erik=20Elfstr=C3=B6m?= Subject: [PATCH v7 1/5] setup: add gentle version of read_gitfile Date: Tue, 9 Jun 2015 20:24:35 +0200 Message-ID: <1433874279-30720-2-git-send-email-erik.elfstrom@gmail.com> References: <1433874279-30720-1-git-send-email-erik.elfstrom@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: =?UTF-8?q?Erik=20Elfstr=C3=B6m?= To: git@vger.kernel.org X-From: git-owner@vger.kernel.org Tue Jun 09 20:25:29 2015 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Z2ODI-0004wS-Qz for gcvg-git-2@plane.gmane.org; Tue, 09 Jun 2015 20:25:29 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753866AbbFISZZ convert rfc822-to-quoted-printable (ORCPT ); Tue, 9 Jun 2015 14:25:25 -0400 Received: from mail-lb0-f196.google.com ([209.85.217.196]:36698 "EHLO mail-lb0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752282AbbFISZX (ORCPT ); Tue, 9 Jun 2015 14:25:23 -0400 Received: by lbiv13 with SMTP id v13so2610379lbi.3 for ; Tue, 09 Jun 2015 11:25:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-type:content-transfer-encoding; bh=pOz0tDNQs8wgdHx+OpWHciatmcCk92GkJvFeyOF55io=; b=dtZx0GNMJEfU/SOOqNo5VttLS2/uwcimQh8IfZYgdhogYIhlgNER0b2wxYt3hFLfyy jQmRszlspiN1cd/DQbJYJugDNrBr79uDT0bcUALetm5sObIVQ4hxne52pkRGnUHUqHCG iVaFg2/Tvr0p6DtPo0B3zDA4GP+ogdI4J7T52TQbLUEqvRusPecpOuFDCtE1F7RwJ0OC pjpAo6nnB+rz8tU+VGwDzKFUgOssscFjNKhLT5T6n5HF3j3VDWKPGGD/49LA4/QyrZyS FiXh0pwyxqEL5gJQkFp5LFTHaWgidRjDUfEL+MH2XVzNQSjkoNSRPnPd4VB2ja2KSS7+ 6JgQ== X-Received: by 10.112.222.133 with SMTP id qm5mr23988181lbc.86.1433874320442; Tue, 09 Jun 2015 11:25:20 -0700 (PDT) Received: from localhost.localdomain (h38n2-lk-d2.ias.bredband.telia.com. [78.72.191.38]) by mx.google.com with ESMTPSA id bm5sm285250lbc.45.2015.06.09.11.25.19 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Tue, 09 Jun 2015 11:25:19 -0700 (PDT) X-Mailer: git-send-email 2.4.3.373.gc496bfb In-Reply-To: <1433874279-30720-1-git-send-email-erik.elfstrom@gmail.com> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: read_gitfile will die on most error cases. This makes it unsuitable for speculative calls. Extract the core logic and provide a gentle version that returns NULL on failure. The first usecase of the new gentle version will be to probe for submodules during git clean. Helped-by: Junio C Hamano Helped-by: Jeff King Signed-off-by: Erik Elfstr=C3=B6m --- cache.h | 11 ++++++++- setup.c | 84 ++++++++++++++++++++++++++++++++++++++++++++++++++-------= -------- 2 files changed, 75 insertions(+), 20 deletions(-) diff --git a/cache.h b/cache.h index 571c98f..25578cb 100644 --- a/cache.h +++ b/cache.h @@ -446,7 +446,16 @@ extern int get_common_dir(struct strbuf *sb, const= char *gitdir); extern const char *get_git_namespace(void); extern const char *strip_namespace(const char *namespaced_ref); extern const char *get_git_work_tree(void); -extern const char *read_gitfile(const char *path); + +#define READ_GITFILE_ERR_STAT_FAILED 1 +#define READ_GITFILE_ERR_NOT_A_FILE 2 +#define READ_GITFILE_ERR_OPEN_FAILED 3 +#define READ_GITFILE_ERR_READ_FAILED 4 +#define READ_GITFILE_ERR_INVALID_FORMAT 5 +#define READ_GITFILE_ERR_NO_PATH 6 +#define READ_GITFILE_ERR_NOT_A_REPO 7 +extern const char *read_gitfile_gently(const char *path, int *return_e= rror_code); +#define read_gitfile(path) read_gitfile_gently((path), NULL) extern const char *resolve_gitdir(const char *suspect); extern void set_git_work_tree(const char *tree); =20 diff --git a/setup.c b/setup.c index 863ddfd..4748b63 100644 --- a/setup.c +++ b/setup.c @@ -406,35 +406,53 @@ static void update_linked_gitdir(const char *gitf= ile, const char *gitdir) /* * Try to read the location of the git directory from the .git file, * return path to git directory if found. + * + * On failure, if return_error_code is not NULL, return_error_code + * will be set to an error code and NULL will be returned. If + * return_error_code is NULL the function will die instead (for most + * cases). */ -const char *read_gitfile(const char *path) +const char *read_gitfile_gently(const char *path, int *return_error_co= de) { - char *buf; - char *dir; + int error_code =3D 0; + char *buf =3D NULL; + char *dir =3D NULL; const char *slash; struct stat st; int fd; ssize_t len; =20 - if (stat(path, &st)) - return NULL; - if (!S_ISREG(st.st_mode)) - return NULL; + if (stat(path, &st)) { + error_code =3D READ_GITFILE_ERR_STAT_FAILED; + goto cleanup_return; + } + if (!S_ISREG(st.st_mode)) { + error_code =3D READ_GITFILE_ERR_NOT_A_FILE; + goto cleanup_return; + } fd =3D open(path, O_RDONLY); - if (fd < 0) - die_errno("Error opening '%s'", path); + if (fd < 0) { + error_code =3D READ_GITFILE_ERR_OPEN_FAILED; + goto cleanup_return; + } buf =3D xmalloc(st.st_size + 1); len =3D read_in_full(fd, buf, st.st_size); close(fd); - if (len !=3D st.st_size) - die("Error reading %s", path); + if (len !=3D st.st_size) { + error_code =3D READ_GITFILE_ERR_READ_FAILED; + goto cleanup_return; + } buf[len] =3D '\0'; - if (!starts_with(buf, "gitdir: ")) - die("Invalid gitfile format: %s", path); + if (!starts_with(buf, "gitdir: ")) { + error_code =3D READ_GITFILE_ERR_INVALID_FORMAT; + goto cleanup_return; + } while (buf[len - 1] =3D=3D '\n' || buf[len - 1] =3D=3D '\r') len--; - if (len < 9) - die("No path in gitfile: %s", path); + if (len < 9) { + error_code =3D READ_GITFILE_ERR_NO_PATH; + goto cleanup_return; + } buf[len] =3D '\0'; dir =3D buf + 8; =20 @@ -448,14 +466,42 @@ const char *read_gitfile(const char *path) free(buf); buf =3D dir; } - - if (!is_git_directory(dir)) - die("Not a git repository: %s", dir); - + if (!is_git_directory(dir)) { + error_code =3D READ_GITFILE_ERR_NOT_A_REPO; + goto cleanup_return; + } update_linked_gitdir(path, dir); path =3D real_path(dir); =20 +cleanup_return: free(buf); + + if (return_error_code) + *return_error_code =3D error_code; + + if (error_code) { + if (return_error_code) + return NULL; + + switch (error_code) { + case READ_GITFILE_ERR_STAT_FAILED: + case READ_GITFILE_ERR_NOT_A_FILE: + return NULL; + case READ_GITFILE_ERR_OPEN_FAILED: + die_errno("Error opening '%s'", path); + case READ_GITFILE_ERR_READ_FAILED: + die("Error reading %s", path); + case READ_GITFILE_ERR_INVALID_FORMAT: + die("Invalid gitfile format: %s", path); + case READ_GITFILE_ERR_NO_PATH: + die("No path in gitfile: %s", path); + case READ_GITFILE_ERR_NOT_A_REPO: + die("Not a git repository: %s", dir); + default: + assert(0); + } + } + return path; } =20 --=20 2.4.3.373.gc496bfb