summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVolker Lendecke <vl@samba.org>2012-04-19 10:20:27 +0200
committerVolker Lendecke <vl@samba.org>2012-04-19 17:37:37 +0200
commit8173331150482b78e85c005fcea2c7d80f430f65 (patch)
treee1c4b606ac7ea91d345d686426f5bba51b3091c6
parentdc3b79626022e07980be40627b9f2255d58b6e70 (diff)
downloadsamba-8173331150482b78e85c005fcea2c7d80f430f65.tar.gz
samba-8173331150482b78e85c005fcea2c7d80f430f65.tar.bz2
samba-8173331150482b78e85c005fcea2c7d80f430f65.zip
s3: Fix Coverity ID 2747: FORWARD_NULL
For the notify cleanup process we have a notify context without a messaging entry. We will never call notify_add/remove for this, but the code should protect against this.
-rw-r--r--source3/smbd/notify_internal.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/source3/smbd/notify_internal.c b/source3/smbd/notify_internal.c
index 27fe21214b..aa02e32a2f 100644
--- a/source3/smbd/notify_internal.c
+++ b/source3/smbd/notify_internal.c
@@ -393,7 +393,7 @@ NTSTATUS notify_remove(struct notify_context *notify, void *private_data)
struct db_record *notify_rec;
NTSTATUS status;
- if (notify == NULL) {
+ if ((notify == NULL) || (notify->msg == NULL)) {
return NT_STATUS_NOT_IMPLEMENTED;
}