Index: ipc/brokerable_attachment.cc |
diff --git a/ipc/brokerable_attachment.cc b/ipc/brokerable_attachment.cc |
index 4a1becc218f57823e9e88af3d9a03a609689cef0..f1cc9b2afaa624b7b61d33b9f83539c159b33779 100644 |
--- a/ipc/brokerable_attachment.cc |
+++ b/ipc/brokerable_attachment.cc |
@@ -4,65 +4,47 @@ |
#include "ipc/brokerable_attachment.h" |
-#include "ipc/attachment_broker.h" |
- |
-#if USE_ATTACHMENT_BROKER |
#include "crypto/random.h" |
-#endif |
namespace IPC { |
-#if USE_ATTACHMENT_BROKER |
-BrokerableAttachment::AttachmentId::AttachmentId() { |
- // In order to prevent mutually untrusted processes from stealing resources |
- // from one another, the nonce must be secret. This generates a 128-bit, |
- // cryptographicaly-strong random number. |
- crypto::RandBytes(nonce, BrokerableAttachment::kNonceSize); |
-} |
-#else |
-BrokerableAttachment::AttachmentId::AttachmentId() { |
- CHECK(false) << "Not allowed to construct an attachment id if the platform " |
- "does not support attachment brokering."; |
-} |
-#endif |
+namespace { |
-BrokerableAttachment::AttachmentId::AttachmentId(const char* start_address, |
- size_t size) { |
- DCHECK(size == BrokerableAttachment::kNonceSize); |
- for (size_t i = 0; i < BrokerableAttachment::kNonceSize; ++i) |
- nonce[i] = start_address[i]; |
+// In order to prevent mutually untrusted processes from stealing resources from |
+// one another, the nonce must be secret. This generates a 128-bit, |
+// cryptographicaly-strong random number. |
+BrokerableAttachment::AttachmentId GetRandomId() { |
+ BrokerableAttachment::AttachmentId id; |
+ crypto::RandBytes(id.nonce, BrokerableAttachment::kNonceSize); |
+ return id; |
} |
-void BrokerableAttachment::AttachmentId::SerializeToBuffer(char* start_address, |
- size_t size) { |
- DCHECK(size == BrokerableAttachment::kNonceSize); |
- for (size_t i = 0; i < BrokerableAttachment::kNonceSize; ++i) |
- start_address[i] = nonce[i]; |
+} // namespace |
+ |
+BrokerableAttachment::BrokerableAttachment() |
+ : id_(GetRandomId()), needs_brokering_(false) {} |
+ |
+BrokerableAttachment::BrokerableAttachment(const AttachmentId& id, |
+ bool needs_brokering) |
+ : id_(id), needs_brokering_(needs_brokering) {} |
+ |
+BrokerableAttachment::~BrokerableAttachment() { |
} |
- |
-BrokerableAttachment::BrokerableAttachment() {} |
- |
-BrokerableAttachment::BrokerableAttachment(const AttachmentId& id) : id_(id) {} |
- |
-BrokerableAttachment::~BrokerableAttachment() {} |
BrokerableAttachment::AttachmentId BrokerableAttachment::GetIdentifier() const { |
return id_; |
} |
bool BrokerableAttachment::NeedsBrokering() const { |
- return GetBrokerableType() == PLACEHOLDER; |
+ return needs_brokering_; |
+} |
+ |
+void BrokerableAttachment::SetNeedsBrokering(bool needs_brokering) { |
+ needs_brokering_ = needs_brokering; |
} |
BrokerableAttachment::Type BrokerableAttachment::GetType() const { |
return TYPE_BROKERABLE_ATTACHMENT; |
} |
-#if defined(OS_POSIX) |
-base::PlatformFile BrokerableAttachment::TakePlatformFile() { |
- NOTREACHED(); |
- return base::PlatformFile(); |
-} |
-#endif // OS_POSIX |
- |
} // namespace IPC |