[NTOS] Remove unnecessary NULL checks and add a missing one. CIDs 1411958, 716670...
authorThomas Faber <thomas.faber@reactos.org>
Sat, 20 Jan 2018 12:42:39 +0000 (13:42 +0100)
committerThomas Faber <thomas.faber@reactos.org>
Sat, 20 Jan 2018 14:59:35 +0000 (15:59 +0100)
ntoskrnl/cc/fs.c
ntoskrnl/dbgk/dbgkobj.c
ntoskrnl/ex/init.c

index d62e18d..4958478 100644 (file)
@@ -337,7 +337,6 @@ CcUninitializeCacheMap (
         FileObject, TruncateSize, UninitializeCompleteEvent);
 
     if (TruncateSize != NULL &&
-        FileObject->SectionObjectPointer != NULL &&
         FileObject->SectionObjectPointer->SharedCacheMap != NULL)
     {
         SharedCacheMap = FileObject->SectionObjectPointer->SharedCacheMap;
index d0f8c85..3f03e9b 100644 (file)
@@ -775,7 +775,7 @@ DbgkpPostFakeThreadMessages(IN PEPROCESS Process,
     {
         /* Dereference and fail */
         if (pFirstThread) ObDereferenceObject(pFirstThread);
-        if (pLastThread) ObDereferenceObject(pLastThread);
+        ObDereferenceObject(pLastThread);
         return Status;
     }
 
index e4ffdd6..e241334 100644 (file)
@@ -1501,7 +1501,7 @@ Phase1InitializationDiscard(IN PVOID Context)
     if (!PoInitSystem(0)) KeBugCheck(INTERNAL_POWER_ERROR);
 
     /* Check for Y2K hack */
-    Y2KHackRequired = strstr(CommandLine, "YEAR");
+    Y2KHackRequired = CommandLine ? strstr(CommandLine, "YEAR") : NULL;
     if (Y2KHackRequired) Y2KHackRequired = strstr(Y2KHackRequired, "=");
     if (Y2KHackRequired) YearHack = atol(Y2KHackRequired + 1);