Catch unable to read memory errors (#739)

* logging failed addresses as hex values instead of base10, sending undefined when unable to read memory

* adjusting to catch error message

---------

Co-authored-by: Thorsten de Buhr <thorsten.deBuhr@arm.com>

Author: omarArm

src/views/component-viewer/component-viewer-target-access.ts

@@ -149,8 +149,10 @@ export class ComponentViewerTargetAccess {
             const response = await this._activeSession?.session.customRequest('readMemory', args) as DebugProtocol.ReadMemoryResponse['body'];
             return response?.data;
         } catch (error: unknown) {
-            const errorMessage = (error as Error)?.message;
-            logger.debug(`Session '${this._activeSession?.session.name}': Failed to read memory at address '${address}' - '${errorMessage}'`);
+            // Change address to hex format for better logging
+            const hexAddress = `0x${Number(address).toString(16)}`;
+            const errorMessage = (error as Error)?.message + ` at address ${hexAddress}`;
+            logger.debug(`Session '${this._activeSession?.session.name}': Failed to read memory at address '${hexAddress}' - '${errorMessage}'`);
             return errorMessage === 'custom request failed' ? 'No active session' : errorMessage;
         }
     }

src/views/component-viewer/scvd-debug-target.ts

@@ -226,6 +226,10 @@ export class ScvdDebugTarget {
         if (typeof dataAsString !== 'string') {
             return undefined;
         }
+        // if data is returned as error message string
+        if (dataAsString.startsWith('Unable')) {
+            return undefined;
+        }
         // Convert String data to Uint8Array
         const byteArray = this.decodeGdbData(dataAsString);