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

Commit 49a8eecc authored by Josh Gao's avatar Josh Gao Committed by Gerrit Code Review
Browse files

Merge "liblog: don't return 0xFFFFFFFF as an invalid log id."

parents 2decb2fc c6ad69d1

liblog/logger_name.cpp

+4 −2
Original line number Diff line number Diff line
@@ -50,8 +50,9 @@ log_id_t android_name_to_log_id(const char* logName) { 
  unsigned int ret;



  if (!logName) {

    return static_cast<log_id_t>(0xFFFFFFFF);

    return static_cast<log_id_t>(LOG_ID_MAX);

  }



  b = strrchr(logName, '/');

  if (!b) {

    b = logName;
@@ -65,5 +66,6 @@ log_id_t android_name_to_log_id(const char* logName) { 
      return static_cast<log_id_t>(ret);

    }

  }

  return static_cast<log_id_t>(0xFFFFFFFF); /* should never happen */



  return static_cast<log_id_t>(LOG_ID_MAX);

}

logcat/tests/logcat_test.cpp

+11 −0
Original line number Diff line number Diff line
@@ -33,6 +33,7 @@ 
#include <android-base/file.h>

#include <android-base/macros.h>

#include <android-base/stringprintf.h>

#include <android-base/strings.h>

#include <gtest/gtest.h>

#include <log/event_tag_map.h>

#include <log/log.h>
@@ -1747,3 +1748,13 @@ TEST(logcat, help) { 
    EXPECT_EQ(logcatHelpTextSize * 2, logcatLastHelpTextSize);

#endif

}



TEST(logcat, invalid_buffer) {

  FILE* fp = popen("logcat -b foo 2>&1", "r");

  ASSERT_NE(nullptr, fp);

  std::string output;

  ASSERT_TRUE(android::base::ReadFdToString(fileno(fp), &output));

  pclose(fp);



  ASSERT_TRUE(android::base::StartsWith(output, "unknown buffer foo\n"));

}