sandbox: avoid an extra if() which is not required and is wrong

sandbox/src/sharedmem_ipc_server.cc

 // Copyright (c) 2011 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 "base/callback.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "sandbox/src/sharedmem_ipc_server.h"
 #include "sandbox/src/sharedmem_ipc_client.h"
 #include "sandbox/src/sandbox.h"
@@ skipping 111 matching lines @@
     return false;
   }
   // This last setting indicates to the client all is setup.
   client_control_->channels_count = channel_count;
   return true;
 }
 
 // Releases memory allocated for IPC arguments, if needed.
 void ReleaseArgs(const IPCParams* ipc_params, void* args[kMaxIpcParams]) {
   for (size_t i = 0; i < kMaxIpcParams; i++) {
-    if (args[i]) {
-      switch (ipc_params->args[i]) {
-        case WCHAR_TYPE: {
-          delete reinterpret_cast<std::wstring*>(args[i]);
-          args[i] = NULL;
-          break;
-        }
-        case INOUTPTR_TYPE: {
-          delete reinterpret_cast<CountedBuffer*>(args[i]);
-          args[i] = NULL;
-          break;
-        }
-        default: break;
+    switch (ipc_params->args[i]) {
+      case WCHAR_TYPE: {
+        delete reinterpret_cast<std::wstring*>(args[i]);
+        args[i] = NULL;
+        break;
       }
+      case INOUTPTR_TYPE: {
+        delete reinterpret_cast<CountedBuffer*>(args[i]);
+        args[i] = NULL;
+        break;
+      }
+      default: break;
     }
   }
 }
 
 // Fills up the list of arguments (args and ipc_params) for an IPC call.
 bool GetArgs(CrossCallParamsEx* params, IPCParams* ipc_params,
              void* args[kMaxIpcParams]) {
   if (kMaxIpcParams < params->GetParamsCount())
     return false;
 
@@ skipping 247 matching lines @@
   }
   *server_pong = ::CreateEventW(NULL, FALSE, FALSE, NULL);
   if (!::DuplicateHandle(::GetCurrentProcess(), *server_pong, target_process_,
                          client_pong, kDesiredAccess, FALSE, 0)) {
     return false;
   }
   return true;
 }
 
 }  // namespace sandbox