From c2b31233b0f33e9171080bc9eb84b15f2648d258 Mon Sep 17 00:00:00 2001 From: Ivan Shapovalov Date: Fri, 17 May 2013 21:12:01 +0400 Subject: Properly check existence of a connection type. Substituting "/dev/null/nonexistent" for an empty $Connection, of course, does generally work, but is somewhat ugly (esp. in that the string is appended to $CONN_DIR, so the resulting path may theoretically exist). Instead, substitute an empty string for the complete path if $Connection is empty: an empty string can never be readable. Signed-off-by: Ivan Shapovalov --- src/lib/globals | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/lib/globals b/src/lib/globals index 1d7feea..83c64a2 100644 --- a/src/lib/globals +++ b/src/lib/globals @@ -109,7 +109,7 @@ load_profile() { if [[ -z $Interface ]]; then exit_error "Profile '$1' does not specify an interface" fi - if [[ ! -r "$CONN_DIR/${Connection:-/dev/null/nonexistent}" ]]; then + if [[ ! -r "${Connection:+$CONN_DIR/$Connection}" ]]; then exit_error "Profile '$1' does not specify a valid connection" fi if [[ -x "$PROFILE_DIR/interfaces/$Interface" ]]; then -- cgit v1.2.3-24-g4f1b