diff options
| author | Jan Cholasta <jcholast@redhat.com> | 2014-11-05 08:50:26 +0000 |
|---|---|---|
| committer | Martin Kosek <mkosek@redhat.com> | 2014-11-05 15:28:27 +0100 |
| commit | 9062dcada48d378f7c46759607aff03c17b11163 (patch) | |
| tree | 83a38cd73e0157ce798085f5db151babf8e7ca52 /daemons/ipa-slapi-plugins/ipa-otp-lasttoken | |
| parent | 701dde3cb3b36173cbe00be5357bcdcbc6dd2b42 (diff) | |
| download | freeipa-9062dcada48d378f7c46759607aff03c17b11163.tar.gz freeipa-9062dcada48d378f7c46759607aff03c17b11163.tar.xz freeipa-9062dcada48d378f7c46759607aff03c17b11163.zip | |
Fix various bugs in ipa-opt-counter and ipa-otp-lasttoken
Fixes a wrong sizeof argument and unchecked return values.
https://fedorahosted.org/freeipa/ticket/4651
Reviewed-By: Alexander Bokovoy <abokovoy@redhat.com>
Diffstat (limited to 'daemons/ipa-slapi-plugins/ipa-otp-lasttoken')
| -rw-r--r-- | daemons/ipa-slapi-plugins/ipa-otp-lasttoken/ipa_otp_lasttoken.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/daemons/ipa-slapi-plugins/ipa-otp-lasttoken/ipa_otp_lasttoken.c b/daemons/ipa-slapi-plugins/ipa-otp-lasttoken/ipa_otp_lasttoken.c index 94d24ae0f..d20fca1e7 100644 --- a/daemons/ipa-slapi-plugins/ipa-otp-lasttoken/ipa_otp_lasttoken.c +++ b/daemons/ipa-slapi-plugins/ipa-otp-lasttoken/ipa_otp_lasttoken.c @@ -44,6 +44,8 @@ #include <libotp.h> #include <time.h> +#include "util.h" + #define PLUGIN_NAME "ipa-otp-lasttoken" #define LOG(sev, ...) \ slapi_log_error(SLAPI_LOG_ ## sev, PLUGIN_NAME, \ @@ -100,7 +102,9 @@ static inline int send_error(Slapi_PBlock *pb, int rc, char *errstr) { slapi_send_ldap_result(pb, rc, NULL, errstr, 0, NULL); - slapi_pblock_set(pb, SLAPI_RESULT_CODE, &rc); + if (slapi_pblock_set(pb, SLAPI_RESULT_CODE, &rc)) { + LOG_FATAL("slapi_pblock_set failed!\n"); + } return rc; } |