From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from simark.ca by simark.ca with LMTP id BtGbH0Kj+1/CBQAAWB0awg (envelope-from ) for ; Sun, 10 Jan 2021 20:00:50 -0500 Received: by simark.ca (Postfix, from userid 112) id 69C5B1EE85; Sun, 10 Jan 2021 20:00:50 -0500 (EST) X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on simark.ca X-Spam-Level: X-Spam-Status: No, score=0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,MAILING_LIST_MULTI,RDNS_NONE,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.2 Received: from sourceware.org (unknown [8.43.85.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPS id EFCA21E940 for ; Sun, 10 Jan 2021 20:00:49 -0500 (EST) Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 50233385480A; Mon, 11 Jan 2021 01:00:49 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 50233385480A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1610326849; bh=5HJRdGgKcgZUv9azr4KMNymxb1lNjWlY2xbyDl0DP4c=; h=To:Subject:Date:In-Reply-To:References:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc: From; b=tSEx/jo+vp1/KWiBYTMPadAJMg9SJkgxa3Bg4MYiuY4TOtAOGBZKPnOGSTyyQ3Ju6 Sd56GDlsZ61cHV5KEVtfsgb4Qu8avgU5X0mjQV3xtWuApdscjGOJfCXgAtxtDRgQcH B2ZADwjEpI7v7mFlLQXLsr9EQZ/F63DQRhyU2F98= Received: from beryx.lancelotsix.com (beryx.lancelotsix.com [164.132.98.193]) by sourceware.org (Postfix) with ESMTPS id 26C36385480A for ; Mon, 11 Jan 2021 01:00:46 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 26C36385480A Received: from gwenhwyvar.lan (unknown [IPv6:2a02:390:8443:0:9f3:4717:3bfb:efbb]) by beryx.lancelotsix.com (Postfix) with ESMTPSA id 06AE92E070; Mon, 11 Jan 2021 02:00:44 +0100 (CET) To: gdb-patches@sourceware.org Subject: [PATCH v3] Improve gdb_tilde_expand logic Date: Mon, 11 Jan 2021 01:00:39 +0000 Message-Id: <20210111010039.23560-1-lsix@lancelotsix.com> X-Mailer: git-send-email 2.29.2 In-Reply-To: <20210108001337.29164-1-lsix@lancelotsix.com> References: <20210108001337.29164-1-lsix@lancelotsix.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.11 (beryx.lancelotsix.com [0.0.0.0]); Mon, 11 Jan 2021 02:00:45 +0100 (CET) X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Lancelot SIX via Gdb-patches Reply-To: Lancelot SIX Cc: Lancelot SIX Errors-To: gdb-patches-bounces@sourceware.org Sender: "Gdb-patches" Before this patch, gdb_tilde_expand would use glob(3) in order to expand tilde at the begining of a path. This implementation has limitation when expanding a tilde leading path to a non existing file since glob fails to expand. This patch proposes to use glob only to expand the tilde component of the path and leaves the rest of the path unchanged. This patch is a followup to the following discution: https://sourceware.org/pipermail/gdb-patches/2021-January/174776.html Before the patch: gdb_tilde_expand("~") -> "/home/lsix" gdb_tilde_expand("~/a/c/b") -> error() is called After the patch: gdb_tilde_expand("~") -> "/home/lsix" gdb_tilde_expand("~/a/c/b") -> "/home/lsix/a/c/b" Tested on x84_64 linux. Since V1: * Unittests added thanks to inputs and skeleton provided by Simon Marchi. * My copyright assignment has been signed! Since V2: * Fix coding style issuess and typos. * Use $HOME and $USER env variables to test the behavior of gdb_tilde_expand instead of testing the function against itself. gdb/ChangeLog: * Makefile.in (SELFTESTS_SRCS): Add unittests/gdb_tilde_expand-selftests.c. * unittests/gdb_tilde_expand-selftests.c: New file. gdbsupport/ChangeLog: * gdb_tilde_expand.cc (gdb_tilde_expand): Improve implementation. (gdb_tilde_expand_up): Delegate logic to gdb_tilde_expand. * gdb_tilde_expand.h (gdb_tilde_expand): Update description. --- gdb/Makefile.in | 1 + gdb/unittests/gdb_tilde_expand-selftests.c | 78 ++++++++++++++++++++++ gdbsupport/gdb_tilde_expand.cc | 39 +++++++---- gdbsupport/gdb_tilde_expand.h | 3 +- 4 files changed, 107 insertions(+), 14 deletions(-) create mode 100644 gdb/unittests/gdb_tilde_expand-selftests.c diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 9267bea7be..c8979fe276 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -446,6 +446,7 @@ SELFTESTS_SRCS = \ unittests/filtered_iterator-selftests.c \ unittests/format_pieces-selftests.c \ unittests/function-view-selftests.c \ + unittests/gdb_tilde_expand-selftests.c \ unittests/gmp-utils-selftests.c \ unittests/lookup_name_info-selftests.c \ unittests/memory-map-selftests.c \ diff --git a/gdb/unittests/gdb_tilde_expand-selftests.c b/gdb/unittests/gdb_tilde_expand-selftests.c new file mode 100644 index 0000000000..d64b7c6384 --- /dev/null +++ b/gdb/unittests/gdb_tilde_expand-selftests.c @@ -0,0 +1,78 @@ +/* Self tests for gdb_tilde_expand + + Copyright (C) 2021 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +#include "gdbsupport/common-defs.h" +#include "gdbsupport/selftest.h" +#include + +#include "gdbsupport/gdb_tilde_expand.h" + +namespace selftests { +namespace gdb_tilde_expand_tests { + +static void +do_test () +{ + /* Home directory of the user running the test. */ + const char *c_home = std::getenv ("HOME"); + + /* Skip the test if $HOME is not available in the environment. */ + if (c_home == nullptr) + return; + + const std::string home (c_home); + + /* Basic situation. */ + SELF_CHECK (home == gdb_tilde_expand ("~")); + + /* When given a path that begins by a tilde and refers to a file that + does not exist, gdb_tilde expand must still be able to do the tilde + expansion. */ + SELF_CHECK (gdb_tilde_expand ("~/non/existent/directory") + == home + "/non/existent/directory"); + + /* gdb_tilde_expand only expands tilde and does not try to do any other + substitution. */ + SELF_CHECK (gdb_tilde_expand ("~/*/a.out") == home + "/*/a.out"); + + /* gdb_tilde_expand does no modification to a non tilde leading path. */ + SELF_CHECK (gdb_tilde_expand ("/some/abs/path") == "/some/abs/path"); + SELF_CHECK (gdb_tilde_expand ("relative/path") == "relative/path"); + + /* If $USER is available in the env variables, Check the '~user' + expansion, otherwise skip the rest of the test. */ + const char *c_user = std::getenv ("USER"); + if (c_user == nullptr) + return; + + const std::string user (c_user); + SELF_CHECK (gdb_tilde_expand (("~" + user).c_str ()) == home); + SELF_CHECK (gdb_tilde_expand (("~" + user + "/a/b").c_str ()) == home + "/a/b"); +} + +} /* namespace gdb_tilde_expand_tests */ +} /* namespace selftests */ + +void _initialize_gdb_tilde_expand_selftests (); +void +_initialize_gdb_tilde_expand_selftests () +{ + selftests::register_test + ("gdb_tilde_expand", selftests::gdb_tilde_expand_tests::do_test); +} diff --git a/gdbsupport/gdb_tilde_expand.cc b/gdbsupport/gdb_tilde_expand.cc index b31fc48446..29224972aa 100644 --- a/gdbsupport/gdb_tilde_expand.cc +++ b/gdbsupport/gdb_tilde_expand.cc @@ -18,6 +18,8 @@ along with this program. If not, see . */ #include "common-defs.h" +#include +#include "filenames.h" #include "gdb_tilde_expand.h" #include @@ -71,14 +73,31 @@ private: std::string gdb_tilde_expand (const char *dir) { - gdb_glob glob (dir, GLOB_TILDE_CHECK, NULL); + if (dir[0] != '~') + return std::string (dir); - gdb_assert (glob.pathc () > 0); - /* "glob" may return more than one match to the path provided by the - user, but we are only interested in the first match. */ - std::string expanded_dir = glob.pathv ()[0]; + /* This function uses glob in order to expand the ~. However, this function + will fail to expand if the actual dir we are looking for does not exist. + Given "~/does/not/exist", glob will fail. - return expanded_dir; + In order to avoid such limitation, we only use glob to expand "~" and keep + "/does/not/exist" unchanged. + + Similarly, to expand ~gdb/might/not/exist, we only expand "~gdb" using + glob and leave "/might/not/exist" unchanged. */ + const std::string d (dir); + + /* Look for the first dir separator (if any) and split d around it. */ + const auto first_sep + = std::find_if (d.cbegin (), d.cend(), + [](const char c) -> bool { return IS_DIR_SEPARATOR (c); }); + const std::string to_expand (d.cbegin (), first_sep); + const std::string remainder (first_sep, d.cend ()); + + const gdb_glob glob (to_expand.c_str (), GLOB_TILDE_CHECK, nullptr); + + gdb_assert (glob.pathc () == 1); + return std::string (glob.pathv ()[0]) + remainder; } /* See gdbsupport/gdb_tilde_expand.h. */ @@ -86,10 +105,6 @@ gdb_tilde_expand (const char *dir) gdb::unique_xmalloc_ptr gdb_tilde_expand_up (const char *dir) { - gdb_glob glob (dir, GLOB_TILDE_CHECK, NULL); - - gdb_assert (glob.pathc () > 0); - /* "glob" may return more than one match to the path provided by the - user, but we are only interested in the first match. */ - return make_unique_xstrdup (glob.pathv ()[0]); + const std::string expanded = gdb_tilde_expand (dir); + return make_unique_xstrdup (expanded.c_str ()); } diff --git a/gdbsupport/gdb_tilde_expand.h b/gdbsupport/gdb_tilde_expand.h index e2d85cadad..a61f246329 100644 --- a/gdbsupport/gdb_tilde_expand.h +++ b/gdbsupport/gdb_tilde_expand.h @@ -20,8 +20,7 @@ #ifndef COMMON_GDB_TILDE_EXPAND_H #define COMMON_GDB_TILDE_EXPAND_H -/* Perform path expansion (i.e., tilde expansion) on DIR, and return - the full path. */ +/* Perform tilde expansion on DIR, and return the full path. */ extern std::string gdb_tilde_expand (const char *dir); /* Same as GDB_TILDE_EXPAND, but return the full path as a -- 2.29.2