commit e4120532bbf3ca9f743b0be1f539381a54d16867
author Lei Zhang <thestig@chromium.org> Wed Jan 15 02:46:57 2014
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Jan 15 23:37:27 2014
tree 1f152b3116219fa682756a73cc00af76fec5900f
parent be12b2f2dd5be1445df39c48088faf47641d872e

dbus-c++: Do not call std::vector::front() on empty vectors.

std::vector will crash in debug mode.

BUG=chromium:334716
TEST=Manual, see bug.

Change-Id: Ib9a3e984e13e5b7d31feaa670642037e8f1b0480
Reviewed-on: https://chromium-review.googlesource.com/182651
Commit-Queue: Lei Zhang <thestig@chromium.org>
Tested-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: mukesh agrawal <quiche@chromium.org>

include/dbus-c++/types.h

diff --git a/include/dbus-c++/types.h b/include/dbus-c++/types.h
index fb73ce5..db7b5d6 100644
--- a/include/dbus-c++/types.h
+++ b/include/dbus-c++/types.h
@@ -296,7 +296,8 @@
 inline DBus::MessageIter &operator << (DBus::MessageIter &iter, const std::vector<uint8_t>& val)
 {
 	DBus::MessageIter ait = iter.new_array("y");
-	ait.append_array('y', &val.front(), val.size());
+	if (!val.empty())
+		ait.append_array('y', &val.front(), val.size());
 	iter.close_container(ait);
 	return iter;
 }