GNU bug report logs - #57208
29.0.50; Uninitialized compilation warnings in xterm.c

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Stefan Kangas <stefankangas <at> gmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#57208: closed (29.0.50; Uninitialized compilation warnings in
 xterm.c)
Date: Tue, 23 Aug 2022 15:28:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Tue, 23 Aug 2022 08:27:14 -0700
with message-id <CADwFkmkYVA4S8+fpjzx_8RrFFUhN3ZmwgP5zpi4TJ37Z7_6G5w <at> mail.gmail.com>
and subject line Re: bug#57208: 29.0.50; Uninitialized compilation warnings in xterm.c
has caused the debbugs.gnu.org bug report #57208,
regarding 29.0.50; Uninitialized compilation warnings in xterm.c
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
57208: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=57208
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: bug-gnu-emacs <at> gnu.org
Subject: 29.0.50; Uninitialized compilation warnings in xterm.c
Date: Sun, 14 Aug 2022 18:57:41 +0300

[Message part 3 (text/plain, inline)]

Severity: minor
Tags: patch

Compiling with GCC 12 and -Og, I get the following -Wmaybe-uninitialized
warnings:

xterm.c: In function ‘x_dnd_free_toplevels’:
xterm.c:2918:17: warning: ‘dpy’ may be used uninitialized [-Wmaybe-uninitialized]
 2918 |       dpyinfo = x_display_info_for_display (dpy);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
xterm.c:2853:12: note: ‘dpy’ was declared here
 2853 |   Display *dpy;
      |            ^~~

xterm.c: In function ‘xm_setup_dnd_targets’:
xterm.c:2284:21: warning: ‘recs’ may be used uninitialized [-Wmaybe-uninitialized]
 2284 |                 recs[header.target_list_count - 1]->targets[i] = targets_sorted[i];
      |                     ^
xterm.c:2112:26: note: ‘recs’ was declared here
 2112 |   xm_targets_table_rec **recs;
      |                          ^~~~

They're bogus, right?  Is it okay to silence them as follows?

[0001-Pacify-maybe-uninitialized-warnings-in-xterm.c.patch (text/x-diff, inline)]

From d14b6fa375ce7a9eded5d0aeae52c8eac893e4de Mon Sep 17 00:00:00 2001
From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
Date: Sun, 14 Aug 2022 18:33:45 +0300
Subject: [PATCH] Pacify maybe-uninitialized warnings in xterm.c

* src/xterm.c (xm_setup_dnd_targets, x_dnd_free_toplevels):
Initialize two pointers as NULL early, to pacify GCC 12
-Wmaybe-uninitialized warnings when compiling with -Og.
---
 src/xterm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/xterm.c b/src/xterm.c
index 6cf44e162b..e7ee5fd32e 100644
--- a/src/xterm.c
+++ b/src/xterm.c
@@ -2109,7 +2109,7 @@ xm_setup_dnd_targets (struct x_display_info *dpyinfo,
   int rc, actual_format, idx;
   bool had_errors;
   xm_targets_table_header header;
-  xm_targets_table_rec **recs;
+  xm_targets_table_rec **recs = NULL;
   xm_byte_order byteorder;
   uint8_t *data;
   ptrdiff_t total_bytes, total_items, i;
@@ -2850,7 +2850,7 @@ x_dnd_free_toplevels (bool display_alive)
   Window *destroy_windows UNINIT;
   unsigned long *prev_masks UNINIT;
   specpdl_ref count;
-  Display *dpy;
+  Display *dpy = NULL;
   struct x_display_info *dpyinfo;
 
   if (!x_dnd_toplevels)
-- 
2.35.1

[Message part 5 (text/plain, inline)]

Thanks,

-- 
Basil

In GNU Emacs 29.0.50 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo version 1.16.0, Xaw3d scroll bars)
 of 2022-08-14 built on tia
Repository revision: 1d3fe256907d5e78a4acedd194e55db8ab952952
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12101004
System Description: Debian GNU/Linux bookworm/sid

Configured using:
 'configure CC=gcc-12 'CFLAGS=-Og -ggdb3' --config-cache
 --prefix=/home/blc/.local --enable-checking=structs
 --with-file-notification=yes --with-x-toolkit=lucid --with-x'

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
JSON LCMS2 LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NOTIFY
INOTIFY PDUMPER PNG RSVG SECCOMP SOUND SQLITE3 THREADS TIFF
TOOLKIT_SCROLL_BARS WEBP X11 XAW3D XDBE XIM XINPUT2 XPM LUCID ZLIB

Important settings:
  value of $LANG: en_IE.UTF-8
  value of $XMODIFIERS: @im=ibus
  locale-coding-system: utf-8-unix

[Message part 6 (message/rfc822, inline)]

From: Stefan Kangas <stefankangas <at> gmail.com>
To: "Basil L. Contovounesios" <contovob <at> tcd.ie>
Cc: Po Lu <luangruo <at> yahoo.com>, 57208-done <at> debbugs.gnu.org
Subject: Re: bug#57208: 29.0.50; Uninitialized compilation warnings in xterm.c
Date: Tue, 23 Aug 2022 08:27:14 -0700

"Basil L. Contovounesios" <contovob <at> tcd.ie> writes:

>
> Po Lu [2022-08-15 08:52 +0800] wrote:
>
>> Yes, those are all bogus.  I installed something different to pacify
>> them on master.
>
> Thanks.

I'm therefore closing this bug report.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.