Added NULL pointer check for DEVMODE.

printing/backend/win_helper.cc

 // Copyright (c) 2012 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 "printing/backend/win_helper.h"
 
 #include <algorithm>
 
 #include "base/file_version_info.h"
 #include "base/files/file_path.h"
@@ skipping 413 matching lines @@
     }
     printing::XPSModule::CloseProvider(provider);
   }
   return scoped_dev_mode.Pass();
 }
 
 scoped_ptr<DEVMODE[]> CreateDevModeWithColor(HANDLE printer,
                                              const base::string16& printer_name,
                                              bool color) {
   scoped_ptr<DEVMODE[]> default = CreateDevMode(printer, NULL);
-  if (default && (default.get()->dmFields & DM_COLOR) &&
+  if (!default)
+    return default.Pass();
+
+  if ((default.get()->dmFields & DM_COLOR) &&
       ((default.get()->dmColor == DMCOLOR_COLOR) == color)) {
     return default.Pass();
   }
 
   default.get()->dmFields |= DM_COLOR;
   default.get()->dmColor = color ? DMCOLOR_COLOR : DMCOLOR_MONOCHROME;
 
   DriverInfo6 info_6;
   if (!info_6.Init(printer))
     return default.Pass();
@@ skipping 28 matching lines @@
   scoped_ptr<DEVMODE[]> out(
       reinterpret_cast<DEVMODE*>(new uint8[buffer_size]));
   flags |= DM_OUT_BUFFER;
   if (DocumentProperties(NULL, printer, L"", out.get(), in, flags) != IDOK)
     return scoped_ptr<DEVMODE[]>();
   DCHECK_EQ(buffer_size, out.get()->dmSize + out.get()->dmDriverExtra);
   return out.Pass();
 }
 
 }  // namespace printing