Improved reproting of kernel return codes.
authormitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 1 Jan 2012 19:24:00 +0000 (19:24 +0000)
committermitz@apple.com <mitz@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 1 Jan 2012 19:24:00 +0000 (19:24 +0000)
Reviewed by Anders Carlsson.

* Platform/CoreIPC/mac/ConnectionMac.cpp:
(CoreIPC::Connection::exceptionSourceEventHandler): Added the error string to the log message.
* Platform/mac/SharedMemoryMac.cpp:
(WebKit::SharedMemory::create): Ditto.
* PluginProcess/mac/PluginProcessMainMac.mm:
(WebKit::PluginProcessMain): Ditto. Also changed to log to stderr.
* WebProcess/mac/WebProcessMainMac.mm:
(WebKit::WebProcessMain): Ditto.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@103879 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/Platform/CoreIPC/mac/ConnectionMac.cpp
Source/WebKit2/Platform/mac/SharedMemoryMac.cpp
Source/WebKit2/PluginProcess/mac/PluginProcessMainMac.mm
Source/WebKit2/WebProcess/mac/WebProcessMainMac.mm

index d739714bb6905c1f29318d9de97c4ee2a4aa5723..5e494559c5964a3a21c0b9e672b18aecc820d2a8 100644 (file)
@@ -1,3 +1,18 @@
+2012-01-01  Dan Bernstein  <mitz@apple.com>
+
+        Improved reproting of kernel return codes.
+
+        Reviewed by Anders Carlsson.
+
+        * Platform/CoreIPC/mac/ConnectionMac.cpp:
+        (CoreIPC::Connection::exceptionSourceEventHandler): Added the error string to the log message.
+        * Platform/mac/SharedMemoryMac.cpp:
+        (WebKit::SharedMemory::create): Ditto.
+        * PluginProcess/mac/PluginProcessMainMac.mm:
+        (WebKit::PluginProcessMain): Ditto. Also changed to log to stderr.
+        * WebProcess/mac/WebProcessMainMac.mm:
+        (WebKit::WebProcessMain): Ditto.
+
 2011-12-31  Dan Bernstein  <mitz@apple.com>
 
         Continue trying to fix the ASSERT-enabled Windows build after r103858.
index 1e5648c743c4648ca13fec3b850ccc78ab465a47..70350d71955d8ca9573539215338260dbfdd05b9 100644 (file)
@@ -404,7 +404,7 @@ void Connection::exceptionSourceEventHandler()
     // Now send along the message.
     kern_return_t kr = mach_msg(header, MACH_SEND_MSG, header->msgh_size, 0, MACH_PORT_NULL, MACH_MSG_TIMEOUT_NONE, MACH_PORT_NULL);
     if (kr != KERN_SUCCESS) {
-        LOG_ERROR("Failed to send message to real exception port, error %x", kr);
+        LOG_ERROR("Failed to send message to real exception port. %s (%x)", mach_error_string(kr), kr);
         ASSERT_NOT_REACHED();
     }
 
index e0d8896eba5a510cf587e9daa4e2523539c97855..291cbd404b4c6346cf99bd57c37ff40e0ced0be8 100644 (file)
@@ -96,7 +96,7 @@ PassRefPtr<SharedMemory> SharedMemory::create(size_t size)
     mach_vm_address_t address;
     kern_return_t kr = mach_vm_allocate(mach_task_self(), &address, round_page(size), VM_FLAGS_ANYWHERE);
     if (kr != KERN_SUCCESS) {
-        LOG_ERROR("Failed to allocate mach_vm_allocate shared memory (%zu bytes) [error code: %x]", size, kr); 
+        LOG_ERROR("Failed to allocate mach_vm_allocate shared memory (%zu bytes). %s (%x)", size, mach_error_string(kr), kr); 
         return 0;
     }
 
@@ -106,7 +106,7 @@ PassRefPtr<SharedMemory> SharedMemory::create(size_t size)
     kr = mach_make_memory_entry_64(mach_task_self(), &memoryObjectSize, address, VM_PROT_DEFAULT, &port, MACH_PORT_NULL);
 
     if (kr != KERN_SUCCESS) {
-        LOG_ERROR("Failed to create a mach port for shared memory [error code: %x]", kr);
+        LOG_ERROR("Failed to create a mach port for shared memory. %s (%x)", mach_error_string(kr), kr);
         mach_vm_deallocate(mach_task_self(), address, round_page(size));
         return 0;
     }
index d4edf5527357579031e59b03780867d8038840e8..abef332c0410316341d1e5f12ac3fd0621ad4364 100644 (file)
@@ -75,7 +75,7 @@ int PluginProcessMain(const CommandLine& commandLine)
     mach_port_t serverPort;
     kern_return_t kr = bootstrap_look_up(bootstrap_port, serviceName.utf8().data(), &serverPort);
     if (kr) {
-        printf("bootstrap_look_up result: %x", kr);
+        fprintf(stderr, "bootstrap_look_up result: %s (%x)", mach_error_string(kr), kr);
         return EXIT_FAILURE;
     }
 
index a80b67cb494aa259af4d44e148765993a7695c65..96bbcca950def35c20b10964eb44ae7445aec39d 100644 (file)
@@ -73,7 +73,7 @@ int WebProcessMain(const CommandLine& commandLine)
     mach_port_t serverPort;
     kern_return_t kr = bootstrap_look_up(bootstrap_port, serviceName.utf8().data(), &serverPort);
     if (kr) {
-        printf("bootstrap_look_up result: %x", kr);
+        fprintf(stderr, "bootstrap_look_up result: %s (%x)", mach_error_string(kr), kr);
         return 2;
     }