From x2go@ymir.das-netzwerkteam.de  Fri Feb 28 16:34:28 2020
Received: (at 1230) by bugs.x2go.org; 28 Feb 2020 15:34:45 +0000
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on
	ymir.das-netzwerkteam.de
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=3.0 tests=BAYES_00,NO_RELAYS,
	URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.2
Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005)
	id 62A605DD18; Fri, 28 Feb 2020 16:34:27 +0100 (CET)
From: =?utf-8?q?Mihai_Moldovan?= <ionic@ionic.de>
To: 1230-submitter@bugs.x2go.org
Cc: control@bugs.x2go.org, 1230@bugs.x2go.org
Subject: X2Go issue (in src:x2goserver) has been marked as pending for release
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
X-Mailer: https://x2go.org/post-receive-tag-pending
Message-Id: <20200228153427.62A605DD18@ymir.das-netzwerkteam.de>
Date: Fri, 28 Feb 2020 16:34:27 +0100 (CET)

tag #1230 pending
fixed #1230 4.0.1.23
thanks

Hi!

X2Go issue #1230 (src:x2goserver) reported by you has been
fixed in X2Go Git. You can see the changelog below, and you can
check the diff of the fix at:

    http://code.x2go.org/gitweb?p=x2goserver.git;a=commitdiff;h=7a76b291

The issue will most likely be fixed in src:x2goserver (4.0.1.23).

light+love
X2Go Git Admin (on behalf of the sender of this mail)

---
commit 7a76b291ed589775ee5d84c34b677fd621ce6d11
Author: Mihai Moldovan <ionic@ionic.de>
Date:   Thu Jan 4 06:16:51 2018 +0100

    x2goserver/bin/x2go{startagent,resume-session}: use x2gogetfreeport instead of duplicating the same code everywhere. Fixes: #1230.
    
    Also, rely on it returning a valid value and try at most 10 times to
    fetch a usable port value, then error out and make the session startup
    or resumption fail.

diff --git a/debian/changelog b/debian/changelog
index 818e823e..f8582d47 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -51,6 +51,11 @@ x2goserver (4.0.1.23-0x2go1) UNRELEASED; urgency=medium
       quotes on the lib path line.
     - x2goserver/bin/x2goresume-session: properly randomize first port, just
       like in x2gostartagent.
+    - x2goserver/bin/x2go{startagent,resume-session}: use x2gogetfreeport
+      instead of duplicating the same code everywhere. Fixes: #1230. Also,
+      rely on it returning a valid value and try at most 10 times to fetch a
+      usable port value, then error out and make the session startup or
+      resumption fail.
   * x2goserver.spec:
     - RPMify x2goserver-xsession description.
     - Remove qt4 stuff, we're not using the framework here.