bugfix for buffer logger type error (#19009)

Buffer impl use `char*` pointer for `Ptr` and `HostPtr`, but that will
confuse the logger and make it consider those as string and try to print
them as string, which will result in SEGV.

Author: yingcong-wu

source/loader/layers/sanitizer/asan/asan_buffer.cpp

@@ -113,7 +113,7 @@ ur_result_t MemBuffer::getHandle(ur_device_handle_t Device, char *&Handle) {
         UR_LOG_L(
             getContext()->logger, ERR,
             "Failed to copy {} bytes data from host pointer {} to buffer {}",
-            Size, HostPtr, this);
+            Size, (void *)HostPtr, this);
         return URes;
       }
     }
@@ -181,7 +181,7 @@ ur_result_t MemBuffer::free() {
     ur_result_t URes = getAsanInterceptor()->releaseMemory(Context, Ptr);
     if (URes != UR_RESULT_SUCCESS) {
       UR_LOG_L(getContext()->logger, ERR, "Failed to free buffer handle {}",
-               Ptr);
+               (void *)Ptr);
       return URes;
     }
   }

source/loader/layers/sanitizer/asan/asan_buffer.hpp

@@ -59,6 +59,7 @@ struct MemBuffer {
 
   size_t Size;
 
+  // FIXME: we should use uint8_t* instead of char* for non-string data.
   char *HostPtr{};
 
   struct SubBuffer_t {

source/loader/layers/sanitizer/msan/msan_buffer.cpp

@@ -153,7 +153,7 @@ ur_result_t MemBuffer::getHandle(ur_device_handle_t Device, char *&Handle) {
         UR_LOG_L(
             getContext()->logger, ERR,
             "Failed to copy {} bytes data from host pointer {} to buffer {}",
-            Size, HostPtr, this);
+            Size, (void *)HostPtr, this);
         return URes;
       }
 
@@ -227,7 +227,7 @@ ur_result_t MemBuffer::free() {
     ur_result_t URes = getContext()->urDdiTable.USM.pfnFree(Context, Ptr);
     if (URes != UR_RESULT_SUCCESS) {
       UR_LOG_L(getContext()->logger, ERR, "Failed to free buffer handle {}",
-               Ptr);
+               (void *)Ptr);
       return URes;
     }
   }

source/loader/layers/sanitizer/msan/msan_buffer.hpp

@@ -59,6 +59,7 @@ struct MemBuffer {
 
   size_t Size;
 
+  // FIXME: we should use uint8_t* instead of char* for non-string data.
   char *HostPtr{};
 
   struct SubBuffer_t {

source/loader/layers/sanitizer/tsan/tsan_buffer.cpp

@@ -121,7 +121,7 @@ ur_result_t MemBuffer::getHandle(ur_device_handle_t Device, char *&Handle) {
         UR_LOG_L(getContext()->logger, ERR,
                  "Failed to copy {} bytes data from host "
                  "pointer {} to buffer {}",
-                 Size, HostPtr, this);
+                 Size, (void *)HostPtr, this);
         return URes;
       }
     }
@@ -189,7 +189,7 @@ ur_result_t MemBuffer::free() {
     ur_result_t URes = getContext()->urDdiTable.USM.pfnFree(Context, Ptr);
     if (URes != UR_RESULT_SUCCESS) {
       UR_LOG_L(getContext()->logger, ERR, "Failed to free buffer handle {}",
-               Ptr);
+               (void *)Ptr);
       return URes;
     }
   }

source/loader/layers/sanitizer/tsan/tsan_buffer.hpp

@@ -59,6 +59,7 @@ struct MemBuffer {
 
   size_t Size;
 
+  // FIXME: we should use uint8_t* instead of char* for non-string data.
   char *HostPtr{};
 
   struct SubBuffer_t {