From 3d9d4558ac5f0e5f34269f37c5e5536bf221eb87 Mon Sep 17 00:00:00 2001
From: "jhawkins@chromium.org"
 <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>
Date: Tue, 6 Oct 2009 19:22:05 +0000
Subject: Coverity: Check the return value of CryptDestroyHash and
 CryptReleaseContext.

CID=6309
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/243103

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28144 0039d316-1c4b-4281-b951-d872f2087c98
---
 chrome/browser/password_manager/ie7_password.cc | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

(limited to 'chrome')

diff --git a/chrome/browser/password_manager/ie7_password.cc b/chrome/browser/password_manager/ie7_password.cc
index a720802..46052f7 100644
--- a/chrome/browser/password_manager/ie7_password.cc
+++ b/chrome/browser/password_manager/ie7_password.cc
@@ -50,10 +50,16 @@ struct PasswordEntry {
 
 // Cleans up a crypt prov and a crypt hash.
 void CleanupHashContext(HCRYPTPROV prov, HCRYPTHASH hash) {
-  if (hash)
-    CryptDestroyHash(hash);
-  if (prov)
-    CryptReleaseContext(prov, 0);
+  BOOL ok;
+  if (hash) {
+    ok = CryptDestroyHash(hash);
+    DCHECK(ok);
+  }
+
+  if (prov) {
+    ok = CryptReleaseContext(prov, 0);
+    DCHECK(ok);
+  }
 }
 
 }  // namespace
-- 
cgit v1.1