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

Commit ef855954 authored by Tom Cherry's avatar Tom Cherry
Browse files

liblog: return -EPERM if a log_is_loggable() returns false 

This was the previous behavior, so retain it.

Bug: 119867234
Test: logging unit tests
Merged-In: I088f760cc274a4a42d72477eb4e2e8d23c613a8a
Change-Id: I088f760cc274a4a42d72477eb4e2e8d23c613a8a
(cherry picked from commit 1ff17fce)
parent aa0ec1a9

liblog/logger_write.cpp

+4 −4
Original line number Diff line number Diff line
@@ -330,7 +330,7 @@ int __android_log_buf_write(int bufID, int prio, const char* tag, const char* ms 
  ErrnoRestorer errno_restorer;



  if (!__android_log_is_loggable(prio, tag, ANDROID_LOG_VERBOSE)) {

    return 0;

    return -EPERM;

  }



  __android_log_message log_message = {
@@ -343,7 +343,7 @@ int __android_log_vprint(int prio, const char* tag, const char* fmt, va_list ap) 
  ErrnoRestorer errno_restorer;



  if (!__android_log_is_loggable(prio, tag, ANDROID_LOG_VERBOSE)) {

    return 0;

    return -EPERM;

  }



  char buf[LOG_BUF_SIZE];
@@ -360,7 +360,7 @@ int __android_log_print(int prio, const char* tag, const char* fmt, ...) { 
  ErrnoRestorer errno_restorer;



  if (!__android_log_is_loggable(prio, tag, ANDROID_LOG_VERBOSE)) {

    return 0;

    return -EPERM;

  }



  va_list ap;
@@ -380,7 +380,7 @@ int __android_log_buf_print(int bufID, int prio, const char* tag, const char* fm 
  ErrnoRestorer errno_restorer;



  if (!__android_log_is_loggable(prio, tag, ANDROID_LOG_VERBOSE)) {

    return 0;

    return -EPERM;

  }



  va_list ap;