mirror of
https://github.com/Divested-Mobile/DivestOS-Build.git
synced 2024-06-26 14:42:13 +00:00
33 lines
1.3 KiB
Diff
33 lines
1.3 KiB
Diff
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
From: Shruti Bihani <shrutibihani@google.com>
|
|
Date: Thu, 6 Jul 2023 08:41:56 +0000
|
|
Subject: [PATCH] Fix Segv on unknown address error flagged by fuzzer test.
|
|
|
|
The error is thrown when the destructor tries to free pointer memory.
|
|
This is happening for cases where the pointer was not initialized. Initializing it to a default value fixes the error.
|
|
|
|
Bug: 245135112
|
|
Test: Build mtp_host_property_fuzzer and run on the target device
|
|
(cherry picked from commit 3afa6e80e8568fe63f893fa354bc79ef91d3dcc0)
|
|
(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:d44311374e41a26b28db56794c9a7890a13a6972)
|
|
Merged-In: I255cd68b7641e96ac47ab81479b9b46b78c15580
|
|
Change-Id: I255cd68b7641e96ac47ab81479b9b46b78c15580
|
|
---
|
|
media/mtp/MtpProperty.h | 3 +++
|
|
1 file changed, 3 insertions(+)
|
|
|
|
diff --git a/media/mtp/MtpProperty.h b/media/mtp/MtpProperty.h
|
|
index 03c08e1eed..45fcbe8917 100644
|
|
--- a/media/mtp/MtpProperty.h
|
|
+++ b/media/mtp/MtpProperty.h
|
|
@@ -24,6 +24,9 @@ namespace android {
|
|
class MtpDataPacket;
|
|
|
|
struct MtpPropertyValue {
|
|
+ // pointer str initialized to NULL so that free operation
|
|
+ // is not called for pre-assigned value
|
|
+ MtpPropertyValue() : str (NULL) {}
|
|
union {
|
|
int8_t i8;
|
|
uint8_t u8;
|