From bc0768c9645251d62d8131987000d5111d84c948 Mon Sep 17 00:00:00 2001 From: Andreas Schneider Date: Thu, 22 Jan 2015 09:50:01 +0100 Subject: [PATCH] s3-pam_smbpass: Fix set_ctrl() return value. This fixes a cppcheck warning. Signed-off-by: Andreas Schneider Reviewed-by: Jeremy Allison --- source3/pam_smbpass/support.c | 5 ++++- source3/pam_smbpass/support.h | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/source3/pam_smbpass/support.c b/source3/pam_smbpass/support.c index 6608c4effb4..f7fec208bc8 100644 --- a/source3/pam_smbpass/support.c +++ b/source3/pam_smbpass/support.c @@ -162,7 +162,10 @@ int make_remark( pam_handle_t * pamh, unsigned int ctrl /* set the control flags for the SMB module. */ -int set_ctrl( pam_handle_t *pamh, int flags, int argc, const char **argv ) +unsigned int set_ctrl(pam_handle_t *pamh, + int flags, + int argc, + const char **argv) { int i = 0; const char *service_file = NULL; diff --git a/source3/pam_smbpass/support.h b/source3/pam_smbpass/support.h index 7ee77042d7f..832247d9985 100644 --- a/source3/pam_smbpass/support.h +++ b/source3/pam_smbpass/support.h @@ -2,7 +2,7 @@ extern void _log_err(pam_handle_t *, int, const char *, ...); /* set the control flags for the UNIX module. */ -extern int set_ctrl(pam_handle_t *, int, int, const char **); +extern unsigned int set_ctrl(pam_handle_t *, int, int, const char **); /* generic function for freeing pam data segments */ extern void _cleanup(pam_handle_t *, void *, int); -- 2.34.1