ScopedProcessInformation protects against process/thread handle leaks from CreateProcess calls.

base/win/scoped_handle.h

Index: base/win/scoped_handle.h
diff --git a/base/win/scoped_handle.h b/base/win/scoped_handle.h
index b89219c45ee3c8fdd6f9a07b3de1ad3b831d9129..d42a5bd64061f137b51f9e38ccd79d4be44e3ebf 100644
--- a/base/win/scoped_handle.h
+++ b/base/win/scoped_handle.h
@@ -41,7 +41,7 @@ class GenericScopedHandle {
   }
 
   void Set(Handle handle) {
-    if (handle_ != handle) {
+    if (!Traits::IsSame(handle_, handle)) {
       Close();
 
       if (Traits::IsHandleValid(handle)) {
@@ -101,6 +101,11 @@ class HandleTraits {
     return handle != NULL && handle != INVALID_HANDLE_VALUE;
   }
 
+  // Returns true if the handle values are the same.
+  static bool IsSame(HANDLE lhs, HANDLE rhs) {

[alexeypa (please no reviews), 2012/03/15 02:41:31]

We will be able to avoid changing all instances of handle traits classes when
making similar changes in the future if we define a base traits class. It should
implement the methods that are usually copy-pasted. Something like this:


template <class T>
class BaseHandleTraits {
 public:
  typedef T Handle;

  // Returns true if the handle value is valid.
  bool IsHandleValid(const T& handle) const {
    return handle != NullHandle();
  }

  bool IsSame(const T& left, const T& right) const {
    return left == right;
  }

  // Returns NULL handle value.
  T NullHandle() const {
    return NULL;
  }
};

class HandleTraits : public BaseHandleTraits<HANDLE> {
 public:

  // Closes the handle.
  bool CloseHandle(HANDLE handle) const {
    return ::CloseHandle(handle) != FALSE;
  }
};

GenericScopedHandle will need to be changed to use non-static methods of the
traits class, which should not be a problem.

[erikwright (departed), 2012/03/15 03:29:17]

I thought about listing this as one of my proposed minor changes. I decided
against it after noting that HandleTraits would only actually use two of these
methods (IsSame and NullHandle).

Another option would be to use a bit of template magic here to use == unless
base::is_class<Handle>
(http://code.google.com/codesearch#OAMlx_jo-ck/src/base/template_util.h&exact_...)
in which case an IsSame method must be provided.

+    return lhs == rhs;
+  }
+
   // Returns NULL handle value.
   static HANDLE NullHandle() {
     return NULL;