Relevant BUGIDs: 848337

Purpose of commit: bugfix

Commit summary:
---------------
pam_shells uses strlen/strcmp, but does not
 include the corresponding string.h header file.
 This could lead to problems on architectures,
 where gcc needs a correct prototype to generate
 correct calling code.

 pam_stress.c complains about a redefined
 __USE_BSD symbol. This is a internal glibc
 define and should not be used in source code.

diff --git a/modules/pam_shells/pam_shells.c b/modules/pam_shells/pam_shells.c
index b70ba030505700fd08c78596ae288154633d0315..37d740dc1b82b9e4df47c45b2a04f12c2650add7 100644 (file)
--- a/modules/pam_shells/pam_shells.c
+++ b/modules/pam_shells/pam_shells.c
@@ -12,6 +12,7 @@
 
 #include <pwd.h>
 #include <stdarg.h>
+#include <string.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <sys/stat.h>

diff --git a/modules/pam_stress/pam_stress.c b/modules/pam_stress/pam_stress.c
index f5126fd21ae3eacb34669d2a1b84e61696af82cc..7828f4214ce8c737bfdafeefa3f941d293e506ba 100644 (file)
--- a/modules/pam_stress/pam_stress.c
+++ b/modules/pam_stress/pam_stress.c
@@ -10,7 +10,6 @@
 #include <stdlib.h>
 #include <stdio.h>
 
-#define __USE_BSD
 #include <syslog.h>
 
 #include <stdarg.h>