From unknown Wed Apr 29 13:10:07 2026 MIME-Version: 1.0 X-Mailer: MIME-tools 5.502 (Entity 5.502) X-Loop: owner@bugs.x2go.org From: owner@bugs.x2go.org (X2Go Bug Tracking System) Subject: Bug#423 closed by Mike Gabriel (X2Go issue (in src:nx-libs) has been marked as closed) Message-ID: References: <20140506160737.9981F3D164@ymir.das-netzwerkteam.de> X-X2go-PR-Keywords: pending patch X-X2go-PR-Message: they-closed 423 X-X2go-PR-Package: nx-libs Date: Tue, 06 May 2014 16:10:07 +0000 Content-Type: multipart/mixed; boundary="----------=_1399392607-21430-0" This is a multi-part message in MIME format... ------------=_1399392607-21430-0 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 This is an automatic notification regarding your Bug report which was filed against the nx-libs package: #423: FIx more -Werror=3Dformat-security errors It has been closed by Mike Gabriel . Their explanation is attached below along with your original report. If this explanation is unsatisfactory and you have not received a better one in a separate message then please contact Mike Gabriel by replying to this email. --=20 X2Go Bug Tracking System Contact owner@bugs.x2go.org with problems ------------=_1399392607-21430-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at control) by bugs.x2go.org; 6 May 2014 16:07:53 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,NO_RELAYS, URIBL_BLOCKED autolearn=ham version=3.3.2 Received: by ymir.das-netzwerkteam.de (Postfix, from userid 1005) id 9981F3D164; Tue, 6 May 2014 18:07:37 +0200 (CEST) From: Mike Gabriel To: 423-submitter@bugs.x2go.org Cc: control@bugs.x2go.org, 423@bugs.x2go.org Subject: X2Go issue (in src:nx-libs) has been marked as closed Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit Message-Id: <20140506160737.9981F3D164@ymir.das-netzwerkteam.de> Date: Tue, 6 May 2014 18:07:37 +0200 (CEST) close #423 thanks Hello, we are very hopeful that X2Go issue #423 reported by you has been resolved in the new release (2:3.5.0.23) of the X2Go source project »src:nx-libs«. You can view the complete changelog entry of src:nx-libs (2:3.5.0.23) below, and you can use the following link to view all the code changes between this and the last release of src:nx-libs. http://code.x2go.org/gitweb?p=nx-libs.git;a=commitdiff;h=fdeaafb5e2f4520857ca4757261df79d233e05cb;hp=626b5dbae6b7b432d573f88abc8e3e8ae2bdba85 If you feel that the issue has not been resolved satisfyingly, feel free to reopen this bug report or submit a follow-up report with further observations described based on the new released version of src:nx-libs. Thanks a lot for contributing to X2Go!!! light+love X2Go Git Admin (on behalf of the sender of this mail) --- X2Go Component: src:nx-libs Version: 2:3.5.0.23-0x2go1 Status: RELEASE Date: Tue, 06 May 2014 18:05:47 +0200 Fixes: 393 411 412 423 Changes: nx-libs (2:3.5.0.23-0x2go1) RELEASED; urgency=low . [ Orion Poplawski ] * Add patch: 054_nx-X11_ppc64-ftbfs.full.patch. Fix FTBFS on ppc64 architecture. (Fixes: #411). * Add patch: 055_nx-X11_imake-Werror-format-security.full.patch. Fix FTBFS of imake on current Fedora rawhide (21) which uses the compiler option -Werror=format-security for all builds. (Fixes: #412). * Add patch: 056_nx-X11-Werror-format-security.patch. Fix more -Werror=format-security errors that only show up on ARM builds. (Fixes: #423). * nx-libs.spec: - Sync nx-libs.spec in Fedora with upstream's nx-libs.spec file. (Fixes: #393). . [ Mike Gabriel ] * debian/control: + Build our imake against x11proto-core-dev headers. * nx-libs.spec: + Build our imake against xorg-x11-proto-devel. . [ Oleksandr Shneyder ] * Add patch: 205_nxagent_refresh-adsl.full.patch. Fix refresh error on RDP connections to Windows 2012 if speed=ADSL. ------------=_1399392607-21430-0 Content-Type: message/rfc822 Content-Disposition: inline Content-Transfer-Encoding: 7bit Received: (at submit) by bugs.x2go.org; 30 Jan 2014 02:27:27 +0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on ymir.das-netzwerkteam.de X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,T_DKIM_INVALID autolearn=ham version=3.3.2 Received: from mail.cora.nwra.com (mercury.cora.nwra.com [4.28.99.165]) by ymir (Postfix) with ESMTPS id E1EEF5DA6C for ; Thu, 30 Jan 2014 03:27:25 +0100 (CET) Received: from pacas.cora.nwra.com (75-171-163-5.hlrn.qwest.net [75.171.163.5]) (authenticated bits=0) by mail.cora.nwra.com (8.14.4/8.14.4) with ESMTP id s0U2RLT2004321 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NO) for ; Wed, 29 Jan 2014 19:27:23 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=cora.nwra.com; s=default; t=1391048844; bh=+c6hJsXCfMloIadt5wvdRx3YHnb0tnaj0xwLoEODXe0=; h=Message-ID:Date:From:MIME-Version:To:Subject:Content-Type; b=cbjKvkYSUgeGq/7UM85DM7fjwAbCwOaQ9eQ6seJQZ6MrNHiH+ZeJSyzyaDAfZ6LeB R577WyAxbgS4OGRsFj/NsJV6cUfv92532A4q563/qCOSOgi8QQEPBtOEZ7OXkD0YRR M3lWoKQMleIsYSM/+U+e2M7ych1lQXuyKIOLR5jA= Message-ID: <52E9B889.8090000@cora.nwra.com> Date: Wed, 29 Jan 2014 19:27:21 -0700 From: Orion Poplawski User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 MIME-Version: 1.0 To: submit@bugs.x2go.org Subject: FIx more -Werror=format-security errors X-Enigmail-Version: 1.6 Content-Type: multipart/mixed; boundary="------------020404070505030607010601" This is a multi-part message in MIME format. --------------020404070505030607010601 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Package: nx-libs Version: 3.5.0.22 Tags: patch The attached patch fixes more -Werror=format-security errors. Interestingly, most of the errors only showed up on our arm builds. No idea why. -- Orion Poplawski Technical Manager 303-415-9701 x222 NWRA/CoRA Division FAX: 303-415-9702 3380 Mitchell Lane orion@cora.nwra.com Boulder, CO 80301 http://www.cora.nwra.com --------------020404070505030607010601 Content-Type: text/x-patch; name="nx-libs-format.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="nx-libs-format.patch" diff -up nx-libs-3.5.0.22/nx-X11/lib/xtrans/Xtransint.h.format nx-libs-3.5.0.22/nx-X11/lib/xtrans/Xtransint.h --- nx-libs-3.5.0.22/nx-X11/lib/xtrans/Xtransint.h.format 2014-01-04 13:39:35.000000000 -0700 +++ nx-libs-3.5.0.22/nx-X11/lib/xtrans/Xtransint.h 2014-01-24 20:29:37.678919812 -0700 @@ -443,7 +443,7 @@ static int trans_mkdir ( int hack= 0, saveerrno=errno; \ struct timeval tp;\ gettimeofday(&tp,0); \ - ErrorF(__xtransname); \ + ErrorF("%s",__xtransname); \ ErrorF(x+hack,a,b,c); \ ErrorF("timestamp (ms): %d\n",tp.tv_sec*1000+tp.tv_usec/1000); \ errno=saveerrno; \ @@ -453,7 +453,7 @@ static int trans_mkdir ( int hack= 0, saveerrno=errno; \ struct timeval tp;\ gettimeofday(&tp,0); \ - fprintf(stderr, __xtransname); fflush(stderr); \ + fprintf(stderr, "%s",__xtransname); fflush(stderr); \ fprintf(stderr, x+hack,a,b,c); fflush(stderr); \ fprintf(stderr, "timestamp (ms): %d\n",tp.tv_sec*1000+tp.tv_usec/1000); \ fflush(stderr); \ @@ -465,14 +465,14 @@ static int trans_mkdir ( /* Use ErrorF() for the X server */ #define PRMSG(lvl,x,a,b,c) if (lvl <= XTRANSDEBUG){ \ int hack= 0, saveerrno=errno; \ - ErrorF(__xtransname); \ + ErrorF("%s",__xtransname); \ ErrorF(x+hack,a,b,c); \ errno=saveerrno; \ } else ((void)0) #else #define PRMSG(lvl,x,a,b,c) if (lvl <= XTRANSDEBUG){ \ int hack= 0, saveerrno=errno; \ - fprintf(stderr, __xtransname); fflush(stderr); \ + fprintf(stderr, "%s",__xtransname); fflush(stderr); \ fprintf(stderr, x+hack,a,b,c); fflush(stderr); \ errno=saveerrno; \ } else ((void)0) diff -up nx-libs-3.5.0.22/nx-X11/programs/nxauth/process.c.format nx-libs-3.5.0.22/nx-X11/programs/nxauth/process.c --- nx-libs-3.5.0.22/nx-X11/programs/nxauth/process.c.format 2014-01-04 13:39:35.000000000 -0700 +++ nx-libs-3.5.0.22/nx-X11/programs/nxauth/process.c 2014-01-28 22:52:26.100107437 -0700 @@ -974,7 +974,7 @@ fprintfhex(register FILE *fp, int len, c char *hex; hex = bintohex(len, cp); - fprintf(fp, hex); + fprintf(fp, "%s", hex); free(hex); } diff -up nx-libs-3.5.0.22/nx-X11/programs/Xserver/GL/glx/glximports.c.format nx-libs-3.5.0.22/nx-X11/programs/Xserver/GL/glx/glximports.c --- nx-libs-3.5.0.22/nx-X11/programs/Xserver/GL/glx/glximports.c.format 2014-01-04 13:39:35.000000000 -0700 +++ nx-libs-3.5.0.22/nx-X11/programs/Xserver/GL/glx/glximports.c 2014-01-27 20:26:27.614602671 -0700 @@ -110,12 +110,12 @@ void *__glXImpRealloc(__GLcontext *gc, v void __glXImpWarning(__GLcontext *gc, char *msg) { - ErrorF((char *)msg); + ErrorF("%s",(char *)msg); } void __glXImpFatal(__GLcontext *gc, char *msg) { - ErrorF((char *)msg); + ErrorF("%s",(char *)msg); __glXAbort(); } diff -up nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Error.c.format nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Error.c --- nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Error.c.format 2014-01-04 13:41:35.000000000 -0700 +++ nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Error.c 2014-01-28 20:35:13.602642690 -0700 @@ -232,7 +232,7 @@ static int nxagentPrintError(dpy, event, int nxagentExitHandler(const char *message) { - FatalError(message); + FatalError("%s", message); return 0; } diff -up nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Init.c.format nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Init.c --- nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Init.c.format 2014-01-04 13:41:35.000000000 -0700 +++ nx-libs-3.5.0.22/nx-X11/programs/Xserver/hw/nxagent/Init.c 2014-01-27 20:53:59.551990127 -0700 @@ -509,7 +509,7 @@ void OsVendorVErrorFFunction(const char nxagentStartRedirectToClientsLog(); - fprintf(stderr, buffer); + fprintf(stderr, "%s", buffer); nxagentEndRedirectToClientsLog(); } diff -up nx-libs-3.5.0.22/nx-X11/programs/Xserver/os/log.c.format nx-libs-3.5.0.22/nx-X11/programs/Xserver/os/log.c --- nx-libs-3.5.0.22/nx-X11/programs/Xserver/os/log.c.format 2014-01-04 13:39:35.000000000 -0700 +++ nx-libs-3.5.0.22/nx-X11/programs/Xserver/os/log.c 2014-01-27 20:01:29.741979120 -0700 @@ -692,9 +692,9 @@ Error(char *str) return; sprintf(err, "%s: ", str); strcat(err, strerror(saveErrno)); - LogWrite(-1, err); + LogWrite(-1, "%s", err); } else - LogWrite(-1, strerror(saveErrno)); + LogWrite(-1, "%s", strerror(saveErrno)); } void --------------020404070505030607010601-- ------------=_1399392607-21430-0--