Sebastian Lackner : server: Add missing check for objattr variable in load_registry wineserver call ( Coverity).

Alexandre Julliard julliard at wine.codeweavers.com
Mon Jan 25 09:27:58 CST 2016


Module: wine
Branch: master
Commit: a52a117f80330e1ad8c322f8282b1eabeafcf0c6
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=a52a117f80330e1ad8c322f8282b1eabeafcf0c6

Author: Sebastian Lackner <sebastian at fds-team.de>
Date:   Sun Jan 24 18:50:43 2016 +0100

server: Add missing check for objattr variable in load_registry wineserver call (Coverity).

Signed-off-by: Sebastian Lackner <sebastian at fds-team.de>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 server/registry.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/server/registry.c b/server/registry.c
index 5723824..a35765c 100644
--- a/server/registry.c
+++ b/server/registry.c
@@ -2187,6 +2187,8 @@ DECL_HANDLER(load_registry)
     const struct security_descriptor *sd;
     const struct object_attributes *objattr = get_req_object_attributes( &sd, &name );
 
+    if (!objattr) return;
+
     if (!thread_single_check_privilege( current, &SeRestorePrivilege ))
     {
         set_error( STATUS_PRIVILEGE_NOT_HELD );




More information about the wine-cvs mailing list