On Mon, Nov 09, 2015 at 12:05:34PM +0100, Lukas Slebodnik wrote:
On (09/11/15 10:30), Jakub Hrozek wrote:
>Hi,
>
>please see the attached trivial patch.
>From a27bf8f594bbbba372a4c324b00b7931306b774c Mon Sep 17 00:00:00 2001
>From: Jakub Hrozek <jhrozek(a)redhat.com>
>Date: Mon, 9 Nov 2015 10:27:41 +0100
>Subject: [PATCH] tools: Don't shadow 'exit'
>MIME-Version: 1.0
>Content-Type: text/plain; charset=UTF-8
>Content-Transfer-Encoding: 8bit
>
>Fixes:
> /sssd/src/tools/sss_override.c: In function ‘override_user_import’:
> /sssd/src/tools/sss_override.c:1471: warning: declaration of ‘exit’ shadows a
global declaration
> /usr/include/stdlib.h:544: warning: shadowed declaration is here
> /sssd/src/tools/sss_override.c: In function ‘override_group_import’:
> /sssd/src/tools/sss_override.c:1737: warning: declaration of ‘exit’ shadows a
global declaration
> /usr/include/stdlib.h:544: warning: shadowed declaration is here
I found few others similar warnings.
Do we want to fix them or ignore?
I can see then only on centos 6 which has old gcc.
See few patches. I might squash them
LS
From 88847d4ec8a06370233301d99f7be7e317acf902 Mon Sep 17 00:00:00
2001
From: Lukas Slebodnik <lslebodn(a)redhat.com>
Date: Mon, 9 Nov 2015 10:40:11 +0100
Subject: [PATCH 1/6] cache_req: Fix warning -Wshadow
ACK
From 2d96b8dd6a6518b7a23bb733108b61665f5a9e39 Mon Sep 17 00:00:00
2001
From: Lukas Slebodnik <lslebodn(a)redhat.com>
Date: Mon, 9 Nov 2015 10:48:55 +0100
Subject: [PATCH 2/6] SBUS: Fix warnings -Wshadow
ACK
From 9cb16f3a5859f6a5f35a908d9b5e6e93b5a75221 Mon Sep 17 00:00:00
2001
From: Lukas Slebodnik <lslebodn(a)redhat.com>
Date: Mon, 9 Nov 2015 10:57:28 +0100
Subject: [PATCH 3/6] SBUS: Fix warnings -Wshadow
Please either squash this patch into the previous one or at least make
them differ in the commit message.
src/sbus/sssd_dbus_signals.c:29: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_signals.c: In function 'sbus_new_incoming_signal':
src/sbus/sssd_dbus_signals.c:39: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_signals.c: In function 'sbus_incoming_signal_hash_add':
src/sbus/sssd_dbus_signals.c:73: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_signals.c: In function 'sbus_incoming_signal_hash_lookup':
src/sbus/sssd_dbus_signals.c:134: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_signals.c: In function 'sbus_signal_listen':
src/sbus/sssd_dbus_signals.c:168: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_signals.c: In function 'sbus_signal_handler':
src/sbus/sssd_dbus_signals.c:227: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_signals.c: In function 'sbus_signal_handler_got_caller_id':
src/sbus/sssd_dbus_signals.c:264: warning: declaration of 'signal' shadows a
global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
src/sbus/sssd_dbus_common_signals.c: In function
'sbus_signal_name_owner_changed':
src/sbus/sssd_dbus_common_signals.c:30: warning: declaration of 'signal' shadows
a global declaration
/usr/include/signal.h:101: warning: shadowed declaration is here
---
src/sbus/sssd_dbus_signals.c | 72 ++++++++++++++++++++++----------------------
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/src/sbus/sssd_dbus_signals.c b/src/sbus/sssd_dbus_signals.c
index 1dc08ae25761881f23a81e7871a8af626497a9a3..3f463e603a625cae8415fb17f5cd811ef0c10e15
100644
--- a/src/sbus/sssd_dbus_signals.c
+++ b/src/sbus/sssd_dbus_signals.c
@@ -26,9 +26,9 @@
#include "sbus/sssd_dbus.h"
#include "sbus/sssd_dbus_private.h"
-static int sbus_incoming_signal_destructor(struct sbus_incoming_signal *signal)
+static int sbus_incoming_signal_destructor(struct sbus_incoming_signal *a_signal)
{
- dbus_message_unref(signal->message);
+ dbus_message_unref(a_signal->message);
return 0;
}
Also please add:
diff --git a/src/sbus/sssd_dbus_common_signals.c b/src/sbus/sssd_dbus_common_signals.c
index 2950c35..959acab 100644
--- a/src/sbus/sssd_dbus_common_signals.c
+++ b/src/sbus/sssd_dbus_common_signals.c
@@ -27,10 +27,10 @@
#include "sbus/sssd_dbus_private.h"
static void
-sbus_signal_name_owner_changed(struct sbus_incoming_signal *signal,
+sbus_signal_name_owner_changed(struct sbus_incoming_signal *sig,
void *handler_data)
{
- hash_table_t *table = signal->conn->clients;
+ hash_table_t *table = sig->conn->clients;
hash_key_t *keys;
unsigned long count;
unsigned long i;
From 97c5be2906005870409a6376a2aa5cfb47284407 Mon Sep 17 00:00:00
2001
From: Lukas Slebodnik <lslebodn(a)redhat.com>
Date: Mon, 9 Nov 2015 10:59:55 +0100
Subject: [PATCH 4/6] test_sss_sifp: Fix warnings -Wshadow
ACK
From 0cb083a55af89eb318b17ca460ce6ebb06d96d58 Mon Sep 17 00:00:00
2001
From: Lukas Slebodnik <lslebodn(a)redhat.com>
Date: Mon, 9 Nov 2015 11:01:22 +0100
Subject: [PATCH 5/6] test_sdap: Fix warnings -Wshadow
src/tests/cmocka/test_sdap.c: In function '__wrap_ldap_next_attribute':
src/tests/cmocka/test_sdap.c:203: warning: declaration of 'index' shadows a
global declaration
/usr/include/string.h:489: warning: shadowed declaration is here
ACK
From 1f8ecc21dc2029b06b310288da9a03719f484182 Mon Sep 17 00:00:00
2001
From: Lukas Slebodnik <lslebodn(a)redhat.com>
Date: Mon, 9 Nov 2015 11:46:18 +0100
Subject: [PATCH 6/6] test_responder_cache_req: Fix warnings -Wshadow
ACK