summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArvid Requate <requate@univention.de>2012-08-14 14:16:54 +0200
committerAndrew Bartlett <abartlet@samba.org>2012-08-14 17:16:54 +0200
commita74ca56c482257f79eb6f966fc490657a0d6407b (patch)
treed5e48d2d9c48424afcffd0b2277bef6c73aae979
parentff5d177a6938a0c486552db362a9e179fbc68b9c (diff)
downloadsamba-a74ca56c482257f79eb6f966fc490657a0d6407b.tar.gz
samba-a74ca56c482257f79eb6f966fc490657a0d6407b.tar.bz2
samba-a74ca56c482257f79eb6f966fc490657a0d6407b.zip
s4:ntp_signd: fix SEGV if SID cannot be found
Signed-off-by: Andrew Bartlett <abartlet@samba.org> Autobuild-User(master): Andrew Bartlett <abartlet@samba.org> Autobuild-Date(master): Tue Aug 14 17:16:54 CEST 2012 on sn-devel-104
-rw-r--r--source4/ntp_signd/ntp_signd.c6
1 files changed, 5 insertions, 1 deletions
diff --git a/source4/ntp_signd/ntp_signd.c b/source4/ntp_signd/ntp_signd.c
index c6d605699f..1de9cc30f9 100644
--- a/source4/ntp_signd/ntp_signd.c
+++ b/source4/ntp_signd/ntp_signd.c
@@ -180,8 +180,12 @@ static NTSTATUS ntp_signd_process(struct ntp_signd_connection *ntp_signd_conn,
}
if (res->count == 0) {
- DEBUG(5, ("Failed to find SID %s in SAM for NTP signing\n",
+ DEBUG(2, ("Failed to find SID %s in SAM for NTP signing\n",
dom_sid_string(mem_ctx, sid)));
+ return signing_failure(ntp_signd_conn,
+ mem_ctx,
+ output,
+ sign_request.packet_id);
} else if (res->count != 1) {
DEBUG(1, ("Found SID %s %u times in SAM for NTP signing\n",
dom_sid_string(mem_ctx, sid), res->count));