commit 153f87a2fd2972f862a88976bb884a90a1f9fd7d
author Steven Moreland <smoreland@google.com> Tue Feb 28 09:42:26 2017 -0800
committer Steven Moreland <smoreland@google.com> Tue Feb 28 09:49:27 2017 -0800
tree c9b47be56f7b3ea9fb97fc7c39efc145d24543a5
parent 311f9dd097a5ffb5d25b245c7442b1270a21df07

Make operator= and constructor behavior consistent.

hidl_string(nullptr) == "". Now so does operator=(nullptr).

Bug: 35421240
Test: libhidl_test
Change-Id: Ib62fa71ab47a407b7017c809783dd5f421b98e39

base/HidlSupport.cpp

diff --git a/base/HidlSupport.cpp b/base/HidlSupport.cpp
index ec5b18f..6f04838 100644
--- a/base/HidlSupport.cpp
+++ b/base/HidlSupport.cpp
@@ -239,6 +239,11 @@
 
 hidl_string &hidl_string::operator=(const char *s) {
     clear();
+
+    if (s == nullptr) {
+        return *this;
+    }
+
     copyFrom(s, strlen(s));
     return *this;
 }