Merge "libbinder Parcel: Fix ubsan error in readData" into main am: 1456474

Treehugger Robot · android-build-merge-worker-robot · commit a346e6940603 · 2024-10-10T04:15:42.000Z
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3300095 Change-Id: If10024d3d45d4f6c75de643e0bcf0b248c6b2415 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/libs/binder/include/binder/Parcel.h b/libs/binder/include/binder/Parcel.h
@@ -1240,7 +1240,7 @@ class Parcel {
             if (__builtin_mul_overflow(size, sizeof(T), &dataLen)) {
                 return -EOVERFLOW;
             }
-            auto data = reinterpret_cast<const T*>(readInplace(dataLen));
+            auto data = readInplace(dataLen);
             if (data == nullptr) return BAD_VALUE;
             // std::vector::insert and similar methods will require type-dependent
             // byte alignment when inserting from a const iterator such as `data`,

Original file line number	Diff line number	Diff line change
`@@ -1240,7 +1240,7 @@ class Parcel {`
`1240`	`1240`	`if (__builtin_mul_overflow(size, sizeof(T), &dataLen)) {`
`1241`	`1241`	`return -EOVERFLOW;`
`1242`	`1242`	`}`
`1243`		`- auto data = reinterpret_cast<const T*>(readInplace(dataLen));`
	`1243`	`+ auto data = readInplace(dataLen);`
`1244`	`1244`	`if (data == nullptr) return BAD_VALUE;`
`1245`	`1245`	`// std::vector::insert and similar methods will require type-dependent`
`1246`	`1246`	// byte alignment when inserting from a const iterator such as `data`,