Loading...
common/Diagnostics.cpp dyld-1340 dyld-1165.3
--- dyld/dyld-1340/common/Diagnostics.cpp
+++ dyld/dyld-1165.3/common/Diagnostics.cpp
@@ -22,7 +22,6 @@
  */
 
 
-#include <limits.h>
 #include <stdint.h>
 #include <stdio.h>
 #include <string.h>
@@ -40,73 +39,26 @@
   #include <iostream>
 #endif
 
-#if !TARGET_OS_EXCLAVEKIT
-#include "BitUtils.h"
-#include <mach/mach.h>
-#endif
-
 #include "Diagnostics.h"
-
-#if !BUILDING_DYLD
-#define _simple_vsnprintf(str, size, fmt, ap) vsnprintf(str, size, fmt, ap)
-#endif
 
 #if BUILDING_CACHE_BUILDER || BUILDING_UNIT_TESTS || BUILDING_CACHE_BUILDER_UNIT_TESTS
   #include <dispatch/dispatch.h>
   dispatch_queue_t sWarningQueue = dispatch_queue_create("com.apple.dyld.cache-builder.warnings", NULL);
 #endif
 
-#if BUILDING_CACHE_BUILDER || BUILDING_UNIT_TESTS || BUILDING_CACHE_BUILDER_UNIT_TESTS
 Diagnostics::Diagnostics(bool verbose)
+#if BUILDING_CACHE_BUILDER || BUILDING_UNIT_TESTS || BUILDING_CACHE_BUILDER_UNIT_TESTS
     : _prefix(""), _verbose(verbose)
-{
-}
-#endif
-
-Diagnostics::Diagnostics()
-{
-#if TARGET_OS_EXCLAVEKIT
-    this->_strBuf[0] = '\0';
-#endif
-}
-
-Diagnostics::Diagnostics(Diagnostics&& other)
-{
-#if TARGET_OS_EXCLAVEKIT
-    strlcpy(this->_strBuf, other._strBuf, sizeof(this->_strBuf));
-    other._strBuf[0] = '\0';
-#else
-    this->_buffer       = other._buffer;
-    this->_bufferSize   = other._bufferSize;
-    this->_bufferUsed   = other._bufferUsed;
-
-    other._buffer       = nullptr;
-    other._bufferSize   = 0;
-    other._bufferUsed   = 0;
-#endif
-}
-
-Diagnostics& Diagnostics::operator=(Diagnostics&& other)
-{
-#if TARGET_OS_EXCLAVEKIT
-    strlcpy(this->_strBuf, other._strBuf, sizeof(this->_strBuf));
-    other._strBuf[0] = '\0';
-#else
-    this->_buffer       = other._buffer;
-    this->_bufferSize   = other._bufferSize;
-    this->_bufferUsed   = other._bufferUsed;
-
-    other._buffer       = nullptr;
-    other._bufferSize   = 0;
-    other._bufferUsed   = 0;
-#endif
-
-    return *this;
+#endif
+{
+#if TARGET_OS_EXCLAVEKIT
+    _strBuf[0] = '\0';
+#endif
 }
 
 #if BUILDING_CACHE_BUILDER || BUILDING_UNIT_TESTS || BUILDING_CACHE_BUILDER_UNIT_TESTS
 Diagnostics::Diagnostics(const std::string& prefix, bool verbose)
-    : _prefix(prefix), _verbose(verbose)
+    : _prefix(prefix),_verbose(verbose)
 {
 }
 #endif
@@ -120,42 +72,18 @@
 {
     va_list    list;
     va_start(list, format);
-    error(format, va_list_wrap(list));
-    va_end(list);
-}
-
-void Diagnostics::error(const char* format, va_list_wrap vaWrap)
-{
-#if TARGET_OS_EXCLAVEKIT
-    vsnprintf(_strBuf, sizeof(_strBuf), format, vaWrap.list);
-#else
-    clearError();
-
-    va_list list2;
-    va_copy(list2, vaWrap.list);
-
-    // Get the size
-    char smallBuffer[1] = { '\0' };
-    int expectedSize = _simple_vsnprintf(smallBuffer, sizeof(smallBuffer), format, vaWrap.list) + 1;
-    assert(expectedSize >= 0);
-
-    const uint64_t alignedSize = lsl::roundToNextAligned(PAGE_SIZE, (uint64_t)expectedSize);
-
-    vm_address_t newBuffer = 0;
-    int kr = ::vm_allocate(mach_task_self(), &newBuffer, (vm_size_t)alignedSize,
-                           VM_FLAGS_ANYWHERE | VM_MAKE_TAG(VM_MEMORY_DYLD));
-    assert(kr == KERN_SUCCESS);
-
-    this->_buffer = (char*)newBuffer;
-
-    int actualSize = _simple_vsnprintf(this->_buffer, (size_t)alignedSize, format, list2);
-    assert(actualSize >= 0);
-    assert((actualSize + 1) == expectedSize);
-
-    this->_bufferUsed = actualSize + 1;
-    this->_bufferSize = (uint32_t)alignedSize;
-
-    va_end(list2);
+    error(format, list);
+    va_end(list);
+}
+
+void Diagnostics::error(const char* format, va_list list)
+{
+#if TARGET_OS_EXCLAVEKIT
+    vsnprintf(_strBuf, sizeof(_strBuf), format, list);
+#else
+    if ( _buffer == nullptr )
+        _buffer = _simple_salloc();
+    _simple_vsprintf(_buffer, format, list);
 #endif
 
 #if BUILDING_CACHE_BUILDER || BUILDING_UNIT_TESTS || BUILDING_CACHE_BUILDER_UNIT_TESTS
@@ -163,26 +91,15 @@
         return;
 
     if (_prefix.empty()) {
-        fprintf(stderr, "%s\n", this->_buffer);
+        fprintf(stderr, "%s\n", _simple_string(_buffer));
     } else {
-        fprintf(stderr, "[%s] %s\n", _prefix.c_str(), this->_buffer);
+        fprintf(stderr, "[%s] %s\n", _prefix.c_str(), _simple_string(_buffer));
     }
 #endif
 }
 
-bool Diagnostics::empty() const
-{
-#if TARGET_OS_EXCLAVEKIT
-    return (this->_strBuf[0] == '\0');
-#else
-    if ( (_buffer == nullptr) || (_bufferUsed == 0) || (_bufferSize == 0) )
-        return true;
-    return false;
-#endif
-}
-
-void Diagnostics::appendError(const char* format, ...)
-{
+ void Diagnostics::appendError(const char* format, ...)
+ {
 #if TARGET_OS_EXCLAVEKIT
     size_t len = strlen(_strBuf);
     va_list list;
@@ -190,63 +107,19 @@
     vsnprintf(&_strBuf[len], sizeof(_strBuf)-len, format, list);
     va_end(list);
 #else
-    if ( empty() ) {
-        va_list list;
-        va_start(list, format);
-        error(format, va_list_wrap(list));
-        va_end(list);
-        return;
-     }
-
-     // Existing buffer, so need to actually append
-     va_list list, list2;
-     va_start(list, format);
-     va_copy(list2, list);
-
-     // Get the size
-     char smallBuffer[1] = { '\0' };
-     int expectedSize = _simple_vsnprintf(smallBuffer, sizeof(smallBuffer), format, list) + 1;
-     assert(expectedSize >= 0);
-
-     uint64_t remainingSize = this->_bufferSize - this->_bufferUsed;
-     if ( remainingSize < expectedSize ) {
-         // Grow the buffer to make space
-         const uint64_t alignedSize = lsl::roundToNextAligned(PAGE_SIZE, (uint64_t)expectedSize + this->_bufferUsed);
-
-         vm_address_t newBuffer = 0;
-         int kr = ::vm_allocate(mach_task_self(), &newBuffer, (vm_size_t)alignedSize,
-                                VM_FLAGS_ANYWHERE | VM_MAKE_TAG(VM_MEMORY_DYLD));
-         assert(kr == KERN_SUCCESS);
-
-         // Copy the old buffer over to the new one
-         memcpy((void*)newBuffer, this->_buffer, this->_bufferSize);
-
-         // Free the existing buffer
-         ::vm_deallocate(mach_task_self(), (vm_address_t)this->_buffer, (vm_size_t)this->_bufferSize);
-
-         // Set the buffer to the new one
-         this->_buffer = (char*)newBuffer;
-         this->_bufferSize = (uint32_t)alignedSize;
-
-         remainingSize = this->_bufferSize - this->_bufferUsed;
-     }
-
-     // Write in to the existing buffer
-     int actualSize = _simple_vsnprintf(&this->_buffer[this->_bufferUsed - 1], (size_t)remainingSize, format, list2);
-     assert(actualSize >= 0);
-     assert((actualSize + 1) == expectedSize);
-
-     this->_bufferUsed += actualSize;
-
-     va_end(list);
-     va_end(list2);
-#endif
-}
+   if ( _buffer != nullptr )
+        _simple_sresize(_buffer);
+    va_list list;
+    va_start(list, format);
+    error(format, list);
+    va_end(list);
+#endif
+ }
 
 bool Diagnostics::hasError() const
 {
 #if TARGET_OS_EXCLAVEKIT
-    return (this->_strBuf[0] != '\0');
+    return (*_strBuf != '\0');
 #else
     return _buffer != nullptr;
 #endif
@@ -255,7 +128,7 @@
 bool Diagnostics::noError() const
 {
 #if TARGET_OS_EXCLAVEKIT
-    return (this->_strBuf[0] == '\0');
+    return (*_strBuf == '\0');
 #else
     return _buffer == nullptr;
 #endif
@@ -264,14 +137,11 @@
 void Diagnostics::clearError()
 {
 #if TARGET_OS_EXCLAVEKIT
-    this->_strBuf[0] = '\0';
-#else
-    if ( _buffer ) {
-        ::vm_deallocate(mach_task_self(), (vm_address_t)this->_buffer, (vm_size_t)this->_bufferSize);
-    }
+    *_strBuf = '\0';
+#else
+    if ( _buffer )
+        _simple_sfree(_buffer);
     _buffer = nullptr;
-    _bufferSize = 0;
-    _bufferUsed = 0;
 #endif
 }
 
@@ -286,11 +156,6 @@
     if ( noError() )
         return false;
     return (strstr(errorMessageCStr(), subString) != nullptr);
-}
-
-std::string_view Diagnostics::errorMessageSV() const
-{
-    return this->errorMessageCStr();
 }
 
 #if BUILDING_CACHE_BUILDER || BUILDING_UNIT_TESTS || BUILDING_CACHE_BUILDER_UNIT_TESTS
@@ -347,7 +212,7 @@
 const char* Diagnostics::errorMessageCStr() const
 {
     if ( _buffer != nullptr )
-        return this->_buffer;
+        return _simple_string(_buffer);
     else
         return "";
 }
@@ -377,7 +242,7 @@
 #if TARGET_OS_EXCLAVEKIT
     return _strBuf;
 #else
-    return this->_buffer;
+    return _simple_string(_buffer);
 #endif
 }
 
@@ -434,21 +299,3 @@
 
 #endif
 
-#if BUILDING_LD || BUILDING_UNIT_TESTS || BUILDING_SHARED_CACHE_LINKER || BUILDING_CACHE_BUILDER || BUILDING_CACHE_BUILDER_UNIT_TESTS
-
-mach_o::Error Diagnostics::toError() const
-{
-    if ( hasError() )
-        return mach_o::Error("%s", errorMessageCStr());
-    return mach_o::Error::none();
-}
-
-void Diagnostics::error(const mach_o::Error& err)
-{
-    if ( err.noError() )
-        return;
-
-    error("%s", err.message());
-}
-
-#endif