Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(737)

Unified Diff: base/file_descriptor_shuffle.cc

Issue 10399100: Move FileDescriptorShuffle to base/posix. This file is very posix-specific. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « base/file_descriptor_shuffle.h ('k') | base/file_descriptor_shuffle_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/file_descriptor_shuffle.cc
diff --git a/base/file_descriptor_shuffle.cc b/base/file_descriptor_shuffle.cc
deleted file mode 100644
index 7ad978729905a9d78a734a89564f56586e2bf706..0000000000000000000000000000000000000000
--- a/base/file_descriptor_shuffle.cc
+++ /dev/null
@@ -1,96 +0,0 @@
-// Copyright (c) 2011 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 "base/file_descriptor_shuffle.h"
-
-#include <unistd.h>
-#include <stddef.h>
-#include <ostream>
-
-#include "base/eintr_wrapper.h"
-#include "base/logging.h"
-
-namespace base {
-
-bool PerformInjectiveMultimapDestructive(
- InjectiveMultimap* m, InjectionDelegate* delegate) {
- static const size_t kMaxExtraFDs = 16;
- int extra_fds[kMaxExtraFDs];
- unsigned next_extra_fd = 0;
-
- // DANGER: this function may not allocate.
-
- for (InjectiveMultimap::iterator i = m->begin(); i != m->end(); ++i) {
- int temp_fd = -1;
-
- // We DCHECK the injectiveness of the mapping.
- for (InjectiveMultimap::iterator j = i + 1; j != m->end(); ++j) {
- DCHECK(i->dest != j->dest) << "Both fd " << i->source
- << " and " << j->source << " map to " << i->dest;
- }
-
- const bool is_identity = i->source == i->dest;
-
- for (InjectiveMultimap::iterator j = i + 1; j != m->end(); ++j) {
- if (!is_identity && i->dest == j->source) {
- if (temp_fd == -1) {
- if (!delegate->Duplicate(&temp_fd, i->dest))
- return false;
- if (next_extra_fd < kMaxExtraFDs) {
- extra_fds[next_extra_fd++] = temp_fd;
- } else {
- RAW_LOG(ERROR, "PerformInjectiveMultimapDestructive overflowed "
- "extra_fds. Leaking file descriptors!");
- }
- }
-
- j->source = temp_fd;
- j->close = false;
- }
-
- if (i->close && i->source == j->dest)
- i->close = false;
-
- if (i->close && i->source == j->source) {
- i->close = false;
- j->close = true;
- }
- }
-
- if (!is_identity) {
- if (!delegate->Move(i->source, i->dest))
- return false;
- }
-
- if (!is_identity && i->close)
- delegate->Close(i->source);
- }
-
- for (unsigned i = 0; i < next_extra_fd; i++)
- delegate->Close(extra_fds[i]);
-
- return true;
-}
-
-bool PerformInjectiveMultimap(const InjectiveMultimap& m_in,
- InjectionDelegate* delegate) {
- InjectiveMultimap m(m_in);
- return PerformInjectiveMultimapDestructive(&m, delegate);
-}
-
-bool FileDescriptorTableInjection::Duplicate(int* result, int fd) {
- *result = HANDLE_EINTR(dup(fd));
- return *result >= 0;
-}
-
-bool FileDescriptorTableInjection::Move(int src, int dest) {
- return HANDLE_EINTR(dup2(src, dest)) != -1;
-}
-
-void FileDescriptorTableInjection::Close(int fd) {
- int ret = HANDLE_EINTR(close(fd));
- DPCHECK(ret == 0);
-}
-
-} // namespace base
« no previous file with comments | « base/file_descriptor_shuffle.h ('k') | base/file_descriptor_shuffle_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698