Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dff58465 authored by Todd Poynor's avatar Todd Poynor
Browse files

reboot: fix owner and permissions of last_reboot_reason file 

Default signature WriteStringToFile creates world-writeable files.
Set owner and group system and remove read/write for non-owner.

Bug: 37251463
Test: Manual: reboot, inspect
Change-Id: I6a29c678168dcae611b120dc52170f4eee7069a9
parent 1c438c1b

init/reboot.cpp

+3 −1
Original line number Diff line number Diff line
@@ -43,6 +43,7 @@ 
#include <cutils/android_reboot.h>

#include <fs_mgr.h>

#include <logwrap/logwrap.h>

#include <private/android_filesystem_config.h>



#include "log.h"

#include "property_service.h"
@@ -305,7 +306,8 @@ void DoReboot(unsigned int cmd, const std::string& reason, const std::string& re 
    Timer t;

    LOG(INFO) << "Reboot start, reason: " << reason << ", rebootTarget: " << rebootTarget;



    android::base::WriteStringToFile(StringPrintf("%s\n", reason.c_str()), LAST_REBOOT_REASON_FILE);

    android::base::WriteStringToFile(StringPrintf("%s\n", reason.c_str()), LAST_REBOOT_REASON_FILE,

                                     S_IRUSR | S_IWUSR, AID_SYSTEM, AID_SYSTEM);



    if (cmd == ANDROID_RB_THERMOFF) {  // do not wait if it is thermal

        DoThermalOff();