[X2Go-Commits] [x2goserver] 01/01: x2goserver/lib/x2go{check, getfree}port: parameter expansion and single quotes don't mix well.

git-admin at x2go.org git-admin at x2go.org
Sun Jan 14 04:43:55 CET 2018


This is an automated email from the git hooks/post-receive script.

x2go pushed a commit to branch master
in repository x2goserver.

commit b1341c2d9aaac8e1383336a2bd2ce3e65f569c86
Author: Mihai Moldovan <ionic at ionic.de>
Date:   Sun Jan 14 04:43:44 2018 +0100

    x2goserver/lib/x2go{check,getfree}port: parameter expansion and single quotes don't mix well.
    
    Use double quotes instead.
---
 debian/changelog               | 2 ++
 x2goserver/lib/x2gocheckport   | 6 +++---
 x2goserver/lib/x2gogetfreeport | 8 ++++----
 3 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 15cbf31..bee48c1 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -228,6 +228,8 @@ x2goserver (4.1.0.0-0x2go1.2) UNRELEASED; urgency=low
       now-unused NX_XINERAMA_CONF variable.
     - x2goserver/lib/x2gogetrandomport: fix function name, leading to startup
       failures.
+    - x2goserver/lib/x2go{check,getfree}port: parameter expansion and single
+      quotes don't mix well. Use double quotes instead.
   * x2goserver.spec:
     - Only create session DB in x2goserver's post install script. Do use
       proper Requires(post) statements to make sure perl-X2Go-Server-DB and
diff --git a/x2goserver/lib/x2gocheckport b/x2goserver/lib/x2gocheckport
index 110766c..9ec668c 100644
--- a/x2goserver/lib/x2gocheckport
+++ b/x2goserver/lib/x2gocheckport
@@ -75,7 +75,7 @@ initialize_x2go_ports() {
 # If an ss command is not given, a default of "ss" is assumed.
 # Returns 0 on success or non-0 on failure.
 initialize_system_ports() {
-	typeset ss="${1:-'ss'}"
+	typeset ss="${1:-"ss"}"
 
 	typeset empty_regex='^[[:space:]]*$'
 	if [[ -z "${ss}" ]] || [[ "${ss}" =~ ${empty_regex} ]]; then
@@ -180,7 +180,7 @@ check_x2go_port() {
 # Returns 0 if the port is free, or non-0 if it already
 # in use or on failure.
 check_system_port() {
-	typeset ss="${1:-'ss'}"
+	typeset ss="${1:-"ss"}"
 	typeset port="${2}"
 
 	typeset empty_regex='^[[:space:]]*$'
@@ -225,7 +225,7 @@ check_system_port() {
 # in use or on failure.
 check_display_port() {
 	typeset current_host_name="${1}"
-	typeset ss="${2:-'ss'}"
+	typeset ss="${2:-"ss"}"
 	typeset port="${3}"
 
 	typeset empty_regex='^[[:space:]]*$'
diff --git a/x2goserver/lib/x2gogetfreeport b/x2goserver/lib/x2gogetfreeport
index d657fda..726ee6e 100755
--- a/x2goserver/lib/x2gogetfreeport
+++ b/x2goserver/lib/x2gogetfreeport
@@ -29,10 +29,10 @@
 # port number on failure.
 # Returns 0 on success or non-0 on failure.
 typeset current_host_name="${1}"
-typeset ss="${2:-'ss'}"
-typeset type="${3:-'lowlevel'}"
-typeset start="${4:-'1'}"
-typeset end="${5:-'65535'}"
+typeset ss="${2:-"ss"}"
+typeset type="${3:-"lowlevel"}"
+typeset start="${4:-"1"}"
+typeset end="${5:-"65535"}"
 
 # Check parameter sanity.
 typeset empty_regex='^[[:space:]]*$'

--
Alioth's /home/x2go-admin/maintenancescripts/git/hooks/post-receive-email on /srv/git/code.x2go.org/x2goserver.git


More information about the x2go-commits mailing list