All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH] setgid03.c: Fix build.
@ 2013-09-12 20:24 Vinson Lee
  2013-09-13 11:22 ` Jan Stancek
  0 siblings, 1 reply; 2+ messages in thread
From: Vinson Lee @ 2013-09-12 20:24 UTC (permalink / raw)
  To: ltp-list

From: Vinson Lee <vlee@twitter.com>

This patch fixes build errors introduced with commit
7dc42c97696f135e1fef92e612ecc66bf8d74318.

setgid03.c:36:23: error: static declaration of ‘getpwnam’ follows non-static declaration
In file included from setgid03.c:24:0:
/usr/include/pwd.h:117:23: note: previous declaration of ‘getpwnam’ was here
setgid03.c:39:1: error: duplicate ‘static’
setgid03.c:40:1: error: duplicate ‘static’

Signed-off-by: Vinson Lee <vlee@twitter.com>
---
 testcases/kernel/syscalls/setgid/setgid03.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/testcases/kernel/syscalls/setgid/setgid03.c b/testcases/kernel/syscalls/setgid/setgid03.c
index 8b71401..8c76556 100644
--- a/testcases/kernel/syscalls/setgid/setgid03.c
+++ b/testcases/kernel/syscalls/setgid/setgid03.c
@@ -33,11 +33,11 @@ int TST_TOTAL = 1;
 
 static char ltpuser1[] = "nobody";
 static char root[] = "root";
-static struct passwd *getpwnam(), *ltpuser1pwent, *rootpwent;
+static struct passwd *ltpuser1pwent, *rootpwent;
 static gid_t mygid;
 
-static static void setup(void);
-static static void cleanup(void);
+static void setup(void);
+static void cleanup(void);
 
 int main(int ac, char **av)
 {
-- 
1.8.1.2


------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. Consolidate legacy IT systems to a single system of record for IT
2. Standardize and globalize service processes across IT
3. Implement zero-touch automation to replace manual, redundant tasks
http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [LTP] [PATCH] setgid03.c: Fix build.
  2013-09-12 20:24 [LTP] [PATCH] setgid03.c: Fix build Vinson Lee
@ 2013-09-13 11:22 ` Jan Stancek
  0 siblings, 0 replies; 2+ messages in thread
From: Jan Stancek @ 2013-09-13 11:22 UTC (permalink / raw)
  To: Vinson Lee, Zhouping Liu; +Cc: ltp-list





----- Original Message -----
> From: "Vinson Lee" <vlee@freedesktop.org>
> To: ltp-list@lists.sourceforge.net
> Sent: Thursday, 12 September, 2013 10:24:39 PM
> Subject: [LTP] [PATCH] setgid03.c: Fix build.
> 
> From: Vinson Lee <vlee@twitter.com>
> 
> This patch fixes build errors introduced with commit
> 7dc42c97696f135e1fef92e612ecc66bf8d74318.
> 
> setgid03.c:36:23: error: static declaration of ‘getpwnam’ follows non-static
> declaration
> In file included from setgid03.c:24:0:
> /usr/include/pwd.h:117:23: note: previous declaration of ‘getpwnam’ was here
> setgid03.c:39:1: error: duplicate ‘static’
> setgid03.c:40:1: error: duplicate ‘static’
> 
> Signed-off-by: Vinson Lee <vlee@twitter.com>

Committed.
Zhouping I added your signed-off-by to this patch as it was identical.

Thank you,
Jan

> ---
>  testcases/kernel/syscalls/setgid/setgid03.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/testcases/kernel/syscalls/setgid/setgid03.c
> b/testcases/kernel/syscalls/setgid/setgid03.c
> index 8b71401..8c76556 100644
> --- a/testcases/kernel/syscalls/setgid/setgid03.c
> +++ b/testcases/kernel/syscalls/setgid/setgid03.c
> @@ -33,11 +33,11 @@ int TST_TOTAL = 1;
>  
>  static char ltpuser1[] = "nobody";
>  static char root[] = "root";
> -static struct passwd *getpwnam(), *ltpuser1pwent, *rootpwent;
> +static struct passwd *ltpuser1pwent, *rootpwent;
>  static gid_t mygid;
>  
> -static static void setup(void);
> -static static void cleanup(void);
> +static void setup(void);
> +static void cleanup(void);
>  
>  int main(int ac, char **av)
>  {
> --
> 1.8.1.2
> 
> 
> ------------------------------------------------------------------------------
> How ServiceNow helps IT people transform IT departments:
> 1. Consolidate legacy IT systems to a single system of record for IT
> 2. Standardize and globalize service processes across IT
> 3. Implement zero-touch automation to replace manual, redundant tasks
> http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
> 

------------------------------------------------------------------------------
How ServiceNow helps IT people transform IT departments:
1. Consolidate legacy IT systems to a single system of record for IT
2. Standardize and globalize service processes across IT
3. Implement zero-touch automation to replace manual, redundant tasks
http://pubads.g.doubleclick.net/gampad/clk?id=51271111&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-09-13 11:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-12 20:24 [LTP] [PATCH] setgid03.c: Fix build Vinson Lee
2013-09-13 11:22 ` Jan Stancek

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.