diff options
author | sleffler@chromium.org <sleffler@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-30 06:46:20 +0000 |
---|---|---|
committer | sleffler@chromium.org <sleffler@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-30 06:46:20 +0000 |
commit | e146bfcf4924815ea32b0e135d720eed26d5bade (patch) | |
tree | 2f6f69341aca38b5f2eb5ec5b0725651f4ea3c3b /dbus/message_unittest.cc | |
parent | 9129b4fc22326a23a141c668e936e650801521e0 (diff) | |
download | chromium_src-e146bfcf4924815ea32b0e135d720eed26d5bade.zip chromium_src-e146bfcf4924815ea32b0e135d720eed26d5bade.tar.gz chromium_src-e146bfcf4924815ea32b0e135d720eed26d5bade.tar.bz2 |
dbus: add support for passing file descriptors
Add support for passing file descriptors in messages.
BUG=chromium-os:27809
TEST=run unit tests
Change-Id: I48e52e52ea1e1a4b96bb0dbec7242337e5871510
Review URL: http://codereview.chromium.org/9700072
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129801 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'dbus/message_unittest.cc')
-rw-r--r-- | dbus/message_unittest.cc | 34 |
1 files changed, 34 insertions, 0 deletions
diff --git a/dbus/message_unittest.cc b/dbus/message_unittest.cc index a40ba0aa..424b50a 100644 --- a/dbus/message_unittest.cc +++ b/dbus/message_unittest.cc @@ -5,6 +5,7 @@ #include "dbus/message.h" #include "base/basictypes.h" +#include "base/eintr_wrapper.h" #include "base/logging.h" #include "base/memory/scoped_ptr.h" #include "dbus/object_path.h" @@ -94,6 +95,39 @@ TEST(MessageTest, AppendAndPopBasicDataTypes) { EXPECT_EQ(dbus::ObjectPath("/object/path"), object_path_value); } +// Check all basic types can be properly written and read. +TEST(MessageTest, AppendAndPopFileDescriptor) { + if (!dbus::kDBusTypeUnixFdIsSupported) { + LOG(WARNING) << "FD passing is not supported"; + return; + } + + scoped_ptr<dbus::Response> message(dbus::Response::CreateEmpty()); + dbus::MessageWriter writer(message.get()); + + // Append stdout. + dbus::FileDescriptor temp(1); + writer.AppendFileDescriptor(temp); + + dbus::FileDescriptor fd_value; + + dbus::MessageReader reader(message.get()); + ASSERT_TRUE(reader.HasMoreData()); + ASSERT_TRUE(reader.PopFileDescriptor(&fd_value)); + ASSERT_FALSE(reader.HasMoreData()); + + // Stdout should be returned but we cannot check the descriptor + // value because stdout will be dup'd. Instead check st_rdev + // which should be identical. + struct stat sb_stdout; + int status_stdout = HANDLE_EINTR(fstat(1, &sb_stdout)); + ASSERT_TRUE(status_stdout >= 0); + struct stat sb_fd; + int status_fd = HANDLE_EINTR(fstat(fd_value.value(), &sb_fd)); + ASSERT_TRUE(status_fd >= 0); + EXPECT_EQ(sb_stdout.st_rdev, sb_fd.st_rdev); +} + // Check all variant types can be properly written and read. TEST(MessageTest, AppendAndPopVariantDataTypes) { scoped_ptr<dbus::Response> message(dbus::Response::CreateEmpty()); |