Restore dump path setting functionality to Linux ExceptionHandler

src/client/linux/handler/minidump_descriptor.cc

 // Copyright (c) 2012 Google Inc.
 // All rights reserved.
 //
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 // notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
@@ skipping 27 matching lines @@
 MinidumpDescriptor::MinidumpDescriptor(const MinidumpDescriptor& descriptor)
     : fd_(descriptor.fd_),
       directory_(descriptor.directory_),
       c_path_(NULL) {
   // The copy constructor is not allowed to be called on a MinidumpDescriptor
   // with a valid path_, as getting its c_path_ would require the heap which
   // can cause problems in compromised environments.
   assert(descriptor.path_.empty());
 }
 
+MinidumpDescriptor& MinidumpDescriptor::operator=(
+    const MinidumpDescriptor& descriptor) {
+  assert(descriptor.path_.empty());
+
+  fd_ = descriptor.fd_;
+  directory_ = descriptor.directory_;
+  path_.clear();
+  c_path_ = NULL;

[digit, 2012/09/13 15:53:59]

Would it make sense to call UpdatePath() here if descriptor.c_path_ is not NULL?

I believe it's slightly cleaner, and would make the test in
ExceptionHandler::set_minidump_descriptor un-necessary.

+  return *this;
+}
+
 void MinidumpDescriptor::UpdatePath() {
   assert(fd_ == -1 && !directory_.empty());
 
   GUID guid;
   char guid_str[kGUIDStringLength + 1];
   bool r = CreateGUID(&guid) && GUIDToString(&guid, guid_str, sizeof(guid_str));
   assert(r);
 
   path_.clear();
   path_ = directory_ + "/" + guid_str + ".dmp";  
   c_path_ = path_.c_str();
 }
 
 }  // namespace google_breakpad