diff options
author | Uri Simchoni <uri@samba.org> | 2017-11-19 18:34:58 +0000 |
---|---|---|
committer | Andrew Bartlett <abartlet@samba.org> | 2017-11-22 10:20:21 +0100 |
commit | 8efe057a8780712373b93cc1b2b55ccc8a44220d (patch) | |
tree | f5ea7bcdf763936827cd6e3bfb8b48f0bb125995 /source3/client | |
parent | 80b4b53fbab4aaab0485411f3c611b66d8a3053b (diff) | |
download | samba-8efe057a8780712373b93cc1b2b55ccc8a44220d.tar.gz |
smbspool_krb5_wrapper: fix some error messages
Make cups_smb_debug declaration printf-aware to
avoid picky warning about printf with variable
format string. This in turn revealed some formatting
errors.
Signed-off-by: Uri Simchoni <uri@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Diffstat (limited to 'source3/client')
-rw-r--r-- | source3/client/smbspool_krb5_wrapper.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/source3/client/smbspool_krb5_wrapper.c b/source3/client/smbspool_krb5_wrapper.c index 9a82b2a7b69..dee3b4c54be 100644 --- a/source3/client/smbspool_krb5_wrapper.c +++ b/source3/client/smbspool_krb5_wrapper.c @@ -37,7 +37,8 @@ enum cups_smb_dbglvl_e { CUPS_SMB_LOG_DEBUG = 0, CUPS_SMB_LOG_ERROR, }; -static void cups_smb_debug(enum cups_smb_dbglvl_e lvl, const char *format, ...); +static void cups_smb_debug(enum cups_smb_dbglvl_e lvl, const char *format, ...) + PRINTF_ATTRIBUTE(2, 3); #define CUPS_SMB_DEBUG(...) cups_smb_debug(CUPS_SMB_LOG_DEBUG, __VA_ARGS__) #define CUPS_SMB_ERROR(...) cups_smb_debug(CUPS_SMB_LOG_DEBUG, __VA_ARGS__) @@ -166,7 +167,7 @@ int main(int argc, char *argv[]) CUPS_SMB_DEBUG("Switching to gid=%d", gid); rc = setgid(gid); if (rc != 0) { - CUPS_SMB_ERROR("Failed to switch to gid=%u", + CUPS_SMB_ERROR("Failed to switch to gid=%u - %s", gid, strerror(errno)); return CUPS_BACKEND_FAILED; @@ -175,7 +176,7 @@ int main(int argc, char *argv[]) CUPS_SMB_DEBUG("Switching to uid=%u", uid); rc = setuid(uid); if (rc != 0) { - CUPS_SMB_ERROR("Failed to switch to uid=%u", + CUPS_SMB_ERROR("Failed to switch to uid=%u - %s", uid, strerror(errno)); return CUPS_BACKEND_FAILED; |