From: Andrew G. Morgan <morgan@kernel.org>
Date: Mon, 12 Nov 2001 07:20:39 +0000 (+0000)
Subject: Relevant BUGIDs: 476938
X-Git-Tag: Linux-PAM-0-76~49
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=6c581ef8dbe547cbc2276355803de4bcddee1f42;p=linux-pam

Relevant BUGIDs: 476938

Purpose of commit: cleanup

Commit summary:
---------------
removed a compiler warning (courtesy Nalin) and added a memory leak comment.
---

diff --git a/CHANGELOG b/CHANGELOG
index 7ab5df55..89d9989a 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -49,6 +49,7 @@ bug report - outstanding bugs are listed here:
 0.76: please submit patches for this section with actual code/doc
       patches!
 
+* pam_motd: Nalin fixed compiler warning. (Bug 476938 - agmorgan)
 * pam_pwdb: Solar Designer pointed out that there was a problem with
   the compatibility support for md5 password hashing. (Bug 460717,
   476961 - agmorgan)
diff --git a/modules/pam_motd/pam_motd.c b/modules/pam_motd/pam_motd.c
index 98976b66..759cc017 100644
--- a/modules/pam_motd/pam_motd.c
+++ b/modules/pam_motd/pam_motd.c
@@ -10,6 +10,8 @@
  *
  */
 
+#include <security/_pam_aconf.h>
+
 #include <stdio.h>
 #include <string.h>
 #include <stdlib.h>
@@ -47,7 +49,8 @@ int pam_sm_open_session(pam_handle_t *pamh, int flags, int argc,
 {
      int retval = PAM_IGNORE;
      int fd;
-     char *mtmp=NULL, *motd_path=NULL;
+     char *mtmp=NULL;
+     const char *motd_path=NULL;
      struct pam_conv *conversation;
      struct pam_message message;
      struct pam_message *pmessage = &message;
@@ -60,9 +63,10 @@ int pam_sm_open_session(pam_handle_t *pamh, int flags, int argc,
 
     for (; argc-- > 0; ++argv) {
         if (!strncmp(*argv,"motd=",5)) {
+
             motd_path = (char *) strdup(5+*argv);
             if (motd_path != NULL) {
-                D(("set motd path: %s", motd_path));
+                D(("set motd path: %s (and a memory leak)", motd_path));
             } else {
                 D(("failed to duplicate motd path - ignored"));
             }