From: Samuel Thibault <samuel.thibault@ens-lyon.org>
Date: Wed, 4 Apr 2018 00:06:16 +0000 (+0200)
Subject: hurd: Silence warning
X-Git-Tag: glibc-2.28~383
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e8ef51b171a79e558651b98be5c7a64c75354098;p=thirdparty%2Fglibc.git

hurd: Silence warning

	* hurd/hurdsig.c (interrupted_reply_port_location): Use
	DIAG_IGNORE_NEEDS_COMMENT to silence warning with GCC 6 and before.
---

diff --git a/ChangeLog b/ChangeLog
index cc9613239de..cbe62ee88d2 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -37,6 +37,8 @@
 
 	* sysdeps/mach/hurd/if_index.c (__if_nametoindex): Return ENODEV if
 	ifname is too long.
+	* hurd/hurdsig.c (interrupted_reply_port_location): Use
+	DIAG_IGNORE_NEEDS_COMMENT to silence warning with GCC 6 and before.
 
 2018-04-03  Wilco Dijkstra  <wdijkstr@arm.com>
 
diff --git a/hurd/hurdsig.c b/hurd/hurdsig.c
index 31e8d336e34..6b73dcdbf38 100644
--- a/hurd/hurdsig.c
+++ b/hurd/hurdsig.c
@@ -33,6 +33,8 @@
 #include "hurdmalloc.h"		/* XXX */
 #include "../locale/localeinfo.h"
 
+#include <libc-diag.h>
+
 const char *_hurdsig_getenv (const char *);
 
 struct mutex _hurd_siglock;
@@ -246,8 +248,14 @@ interrupted_reply_port_location (thread_t thread,
     /* Faulted trying to read the TCB.  */
     return NULL;
 
+  DIAG_PUSH_NEEDS_COMMENT;
+  /* GCC 6 and before seem to be confused by the setjmp call inside
+     _hurdsig_catch_memory_fault and think that we may be returning a second
+     time to here with portloc uninitialized (but we never do). */
+  DIAG_IGNORE_NEEDS_COMMENT (6, "-Wmaybe-uninitialized");
   /* Fault now if this pointer is bogus.  */
   *(volatile mach_port_t *) portloc = *portloc;
+  DIAG_POP_NEEDS_COMMENT;
 
   if (sigthread)
     _hurdsig_end_catch_fault ();