commit 171a64a823952b0ab210688a1d342e9882078400
author Frederick Mayle <fmayle@google.com> Fri Mar 25 22:44:39 2022 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Fri Mar 25 22:44:39 2022 +0000
tree ee49e81e5fc27c667abd1b8dbbd49b393c90a33d
parent 9146157b642fb149697dfea3727f64ad27a9fe8f
parent e658d330c5c2aac5682e528bef3a373dc21b7aa9

Merge "binder: Don't use reinterpret_cast to down cast"

libs/binder/include/binder/ParcelableHolder.h

diff --git a/libs/binder/include/binder/ParcelableHolder.h b/libs/binder/include/binder/ParcelableHolder.h
index 42c85f9..88790a8 100644
--- a/libs/binder/include/binder/ParcelableHolder.h
+++ b/libs/binder/include/binder/ParcelableHolder.h
@@ -86,7 +86,7 @@
                 *ret = nullptr;
                 return android::BAD_VALUE;
             }
-            *ret = std::shared_ptr<T>(mParcelable, reinterpret_cast<T*>(mParcelable.get()));
+            *ret = std::static_pointer_cast<T>(mParcelable);
             return android::OK;
         }
         this->mParcelPtr->setDataPosition(0);
@@ -105,7 +105,7 @@
             return status;
         }
         this->mParcelPtr = nullptr;
-        *ret = std::shared_ptr<T>(mParcelable, reinterpret_cast<T*>(mParcelable.get()));
+        *ret = std::static_pointer_cast<T>(mParcelable);
         return android::OK;
     }