Cherry pick PR #2205: Fix uninitialized variable append_ in class con…

…structors (#2358) Refer to the original PR: #2205 b/219048469 Test-On-Device: true Co-authored-by: alexanderbobrovnik <[email protected]>
youtube · Feb 7, 2024 · 2b106b7 · 2b106b7
1 parent 9209c19
commit 2b106b7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/base/files/file.cc b/base/files/file.cc
@@ -64,7 +64,13 @@ File::File(File&& other)
       tracing_path_(other.tracing_path_),
       error_details_(other.error_details()),
       created_(other.created()),
-      async_(other.async_) {}
+      async_(other.async_)
+#if defined(STARBOARD)
+      ,
+      append_(other.append_)
+#endif
+{
+}
 
 File::~File() {
   // Go through the AssertIOAllowed logic.
@@ -78,6 +84,9 @@ File& File::operator=(File&& other) {
   error_details_ = other.error_details();
   created_ = other.created();
   async_ = other.async_;
+#if defined(STARBOARD)
+  append_ = other.append_;
+#endif
   return *this;
 }
 

diff --git a/base/files/file.h b/base/files/file.h
@@ -380,7 +380,7 @@ class BASE_EXPORT File {
   bool async_;
 
 #if defined(STARBOARD)
-  bool append_;
+  bool append_ = false;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(File);