Remove a DCHECK when failing to secure the shared memory used to hold the cur...

chrome_frame/module_utils.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome_frame/module_utils.h"
 
 #include <aclapi.h>
 #include <atlbase.h>
 #include <atlsecurity.h>
 #include <sddl.h>
@@ skipping 130 matching lines @@
   }
 
   bool created_beacon = true;
   bool result = shared_memory_->CreateNamed(shared_memory_name_.c_str(),
                                             false,  // open_existing
                                             kSharedMemorySize);
 
   if (result) {
     // We created the beacon, now we need to mutate the security attributes
     // on the shared memory to allow read-only access and let low-integrity
-    // processes open it.
-    bool acls_set = SetFileMappingToReadOnly(shared_memory_->handle());
-    DCHECK(acls_set);
+    // processes open it. This will fail on FAT32 file systems.
+    if (!SetFileMappingToReadOnly(shared_memory_->handle())) {
+      DLOG(ERROR) << "Failed to set file mapping permissions.";
+    }
   } else {
     created_beacon = false;
 
     // We failed to create the shared memory segment, suggesting it may already
     // exist: try to create it read-only.
     result = shared_memory_->Open(shared_memory_name_.c_str(),
                                   true /* read_only */);
   }
 
   if (result) {
@@ skipping 112 matching lines @@
 
   HMODULE hmodule = LoadLibrary(module_path.value().c_str());
   if (hmodule == NULL) {
     DPLOG(ERROR) << "Could not load reported module version "
                  << version->GetString();
   }
 
   return hmodule;
 }