[src/trunk]: src/dist/openpam/include/security Apply r432 from openpam reposi...

[joerg <joerg%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/014a278a0895
branches:  trunk
changeset: 753183:014a278a0895
user:      joerg <joerg%NetBSD.org@localhost>
date:      Fri Mar 19 18:16:14 2010 +0000

description:
Apply r432 from openpam repository:
pam_end() already contains a NULL check, and it is not unreasonable to
call it with a NULL pamh in a cleanup / error-handling situation. Remove
OPENPAM_NONNULL, which may cause gcc to optimize away the NULL check.

diffstat:

 dist/openpam/include/security/pam_appl.h |  5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diffs (22 lines):

diff -r 97aefa82ef68 -r 014a278a0895 dist/openpam/include/security/pam_appl.h
--- a/dist/openpam/include/security/pam_appl.h  Fri Mar 19 18:11:30 2010 +0000
+++ b/dist/openpam/include/security/pam_appl.h  Fri Mar 19 18:16:14 2010 +0000
@@ -32,7 +32,7 @@
  * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
  * SUCH DAMAGE.
  *
- * $Id: pam_appl.h,v 1.1.1.2 2008/01/27 00:54:59 christos Exp $
+ * $Id: pam_appl.h,v 1.2 2010/03/19 18:16:14 joerg Exp $
  */
 
 #ifndef SECURITY_PAM_APPL_H_INCLUDED
@@ -72,8 +72,7 @@
 
 int
 pam_end(pam_handle_t *_pamh,
-       int _status)
-       OPENPAM_NONNULL((1));
+       int _status);
 
 int
 pam_get_data(const pam_handle_t *_pamh,