Slight tweak to test logic and fix the printf for mismatched addresses

Author: Jlalond

lldb/source/Plugins/ObjectFile/Minidump/MinidumpFileBuilder.cpp

@@ -992,7 +992,7 @@ Status MinidumpFileBuilder::ReadWriteMemoryInChunks(
       // we'll error out of this Minidump generation.
       addDataError = Status::FromErrorStringWithFormat(
           "Unexpected address encounterd when reading memory in chunks "
-          "0x" PRIx64 " expected 0x" PRIx64,
+          "0x%" PRIx64 " expected 0x%" PRIx64,
           current_addr, addr + total_bytes_read);
       return lldb_private::IterationAction::Stop;
     }

lldb/test/API/functionalities/process_save_core_minidump/TestProcessSaveCoreMinidump64b.py

@@ -60,15 +60,10 @@ def verify_minidump(
                     )
 
                 # Now we check if the error is the same, error isn't abnormal but they should fail for the same reason
+                # Success will be false if they both fail
                 self.assertTrue(
-                    (
-                        actual_process_read_error.Success()
-                        and expected_process_read_error.Success()
-                    )
-                    or (
-                        actual_process_read_error.Fail()
-                        and expected_process_read_error.Fail()
-                    ),
+                    actual_process_read_error.Success()
+                    == expected_process_read_error.Success(),
                     f"Address range {hex(start_addr)} - {hex(end_addr)} failed to read from live process and core for different reasons",
                 )
         finally: