Revert "libc: Add logcat error message for memory corruption"
This fixes the build.
This reverts commit 7708a89c60e7b024d31c48c8034932c5e9f0aceb.
diff --git a/libc/bionic/dlmalloc.c b/libc/bionic/dlmalloc.c
index 035dbac..19fbb75 100644
--- a/libc/bionic/dlmalloc.c
+++ b/libc/bionic/dlmalloc.c
@@ -2265,53 +2265,13 @@
#else /* PROCEED_ON_ERROR */
-/* The following Android-specific code is used to print an informative
- * fatal error message to the log when we detect that a heap corruption
- * was detected. We need to be careful about not using a log function
- * that may require an allocation here!
- */
-#ifdef __ANDROID__
+#ifndef CORRUPTION_ERROR_ACTION
+#define CORRUPTION_ERROR_ACTION(m) ABORT
+#endif /* CORRUPTION_ERROR_ACTION */
-# include <private/logd.h>
-
-static void __bionic_heap_error(const char* msg, const char* function)
-{
- /* We format the buffer explicitely, i.e. without using snprintf()
- * which may use malloc() internally. Not something we can trust
- * if we just detected a corrupted heap.
- */
- char buffer[256];
- strlcpy(buffer, "@@@ ABORTING: ", sizeof(buffer));
- strlcat(buffer, msg, sizeof(buffer));
- if (function != NULL) {
- strlcat(buffer, " IN ", sizeof(buffer));
- strlcat(buffer, function, sizeof(buffer));
- }
- __libc_android_log_write(ANDROID_LOG_FATAL,"libc","%s", buffer);
- abort();
-}
-
-# ifndef CORRUPTION_ERROR_ACTION
-# define CORRUPTION_ERROR_ACTION(m) \
- __bionic_heap_error("HEAP MEMORY CORRUPTION", __FUNCTION__)
-# endif
-# ifndef USAGE_ERROR_ACTION
-# define USAGE_ERROR_ACTION(m,p) \
- __bionic_heap_error("INVALID HEAP ADDRESS", __FUNCTION__)
-# endif
-
-#else /* !__ANDROID__ */
-
-# ifndef CORRUPTION_ERROR_ACTION
-# define CORRUPTION_ERROR_ACTION(m) ABORT
-# endif /* CORRUPTION_ERROR_ACTION */
-
-# ifndef USAGE_ERROR_ACTION
-# define USAGE_ERROR_ACTION(m,p) ABORT
-# endif /* USAGE_ERROR_ACTION */
-
-#endif /* !__ANDROID__ */
-
+#ifndef USAGE_ERROR_ACTION
+#define USAGE_ERROR_ACTION(m,p) ABORT
+#endif /* USAGE_ERROR_ACTION */
#endif /* PROCEED_ON_ERROR */
diff --git a/libc/bionic/logd_write.c b/libc/bionic/logd_write.c
index 2bc39fa..63dfd59 100644
--- a/libc/bionic/logd_write.c
+++ b/libc/bionic/logd_write.c
@@ -48,16 +48,6 @@
#include <pthread.h>
-/* IMPORTANT IMPORTANT IMPORTANT: TECHNICAL NOTE
- *
- * Some of the functions below can be called when our malloc() implementation
- * has detected that the heap is corrupted, or even from a signal handler.
- *
- * These functions should *not* use a function that allocates heap memory
- * or is not signal-safe. Using direct system calls is acceptable, and we
- * also assume that pthread_mutex_lock/unlock can be used too.
- */
-
#define LOG_BUF_SIZE 1024
typedef enum {
@@ -87,10 +77,9 @@
{ __write_to_log_init, -1, "/dev/"LOGGER_LOG_RADIO }
};
-/* Important: see technical note at start of source file */
static int __write_to_log_null(log_id_t log_id, struct iovec *vec)
{
- /*
+ /*
* ALTERED behaviour from previous version
* always returns successful result
*/
@@ -108,21 +97,23 @@
* it's supposed, that log_id contains valid id always.
* this check must be performed in higher level functions
*/
-/* Important: see technical note at start of source file */
static int __write_to_log_kernel(log_id_t log_id, struct iovec *vec)
{
- return TEMP_FAILURE_RETRY( writev(log_channels[log_id].fd, vec, 3) );
+ ssize_t ret;
+
+ do {
+ ret = writev(log_channels[log_id].fd, vec, 3);
+ } while ((ret < 0) && (errno == EINTR));
+
+ return ret;
}
-/* Important: see technical note at start of source file */
static int __write_to_log_init(log_id_t log_id, struct iovec *vec)
{
if ((LOG_ID_NONE < log_id) && (log_id < LOG_ID_MAX)) {
- int fd;
-
pthread_mutex_lock(&log_init_lock);
- fd = TEMP_FAILURE_RETRY(open(log_channels[log_id].path, O_WRONLY));
+ int fd = open(log_channels[log_id].path, O_WRONLY);
log_channels[log_id].logger =
(fd < 0) ? __write_to_log_null : __write_to_log_kernel;
@@ -139,9 +130,7 @@
return -1;
}
-/* Important: see technical note at start of source file */
-__LIBC_HIDDEN__
-int __libc_android_log_write(int prio, const char *tag, const char *msg)
+static int __android_log_write(int prio, const char *tag, const char *msg)
{
struct iovec vec[3];
log_id_t log_id = LOG_ID_MAIN;
@@ -162,11 +151,7 @@
return log_channels[log_id].logger(log_id, vec);
}
-/* The functions below are not designed to be called from a heap panic
- * function or from a signal handler. As such, they are free to use complex
- * C library functions like vsnprintf()
- */
-__LIBC_HIDDEN__
+
int __libc_android_log_vprint(int prio, const char *tag, const char *fmt,
va_list ap)
{
@@ -174,10 +159,9 @@
vsnprintf(buf, LOG_BUF_SIZE, fmt, ap);
- return __libc_android_log_write(prio, tag, buf);
+ return __android_log_write(prio, tag, buf);
}
-__LIBC_HIDDEN__
int __libc_android_log_print(int prio, const char *tag, const char *fmt, ...)
{
va_list ap;
@@ -187,21 +171,20 @@
vsnprintf(buf, LOG_BUF_SIZE, fmt, ap);
va_end(ap);
- return __libc_android_log_write(prio, tag, buf);
+ return __android_log_write(prio, tag, buf);
}
-__LIBC_HIDDEN__
int __libc_android_log_assert(const char *cond, const char *tag,
const char *fmt, ...)
{
va_list ap;
- char buf[LOG_BUF_SIZE];
+ char buf[LOG_BUF_SIZE];
va_start(ap, fmt);
vsnprintf(buf, LOG_BUF_SIZE, fmt, ap);
va_end(ap);
- __libc_android_log_write(ANDROID_LOG_FATAL, tag, buf);
+ __android_log_write(ANDROID_LOG_FATAL, tag, buf);
exit(1);
diff --git a/libc/private/logd.h b/libc/private/logd.h
index 4a9b62e..43fa742 100644
--- a/libc/private/logd.h
+++ b/libc/private/logd.h
@@ -44,7 +44,6 @@
ANDROID_LOG_SILENT, /* only for SetMinPriority(); must be last */
};
-int __libc_android_log_write(int prio, const char* tag, const char* buffer);
int __libc_android_log_print(int prio, const char *tag, const char *fmt, ...);
int __libc_android_log_vprint(int prio, const char *tag, const char *fmt, va_list ap);