[SG-720] Trim c null characters getting padded at end of messages

apps/desktop/src/services/nativeMessageHandler.service.ts

@@ -182,12 +182,25 @@ export class NativeMessageHandlerService {
       this.ddgSharedSecret = SymmetricCryptoKey.fromJSON({ keyB64: storedKey });
     }
 
-    return JSON.parse(
-      await this.cryptoService.decryptToUtf8(
+    try {
+      let decryptedResult = await this.cryptoService.decryptToUtf8(
         message.encryptedCommand as EncString,
         this.ddgSharedSecret
-      )
-    );
+      );
+
+      decryptedResult = this.trimNullCharsFromMessage(decryptedResult);
+
+      return JSON.parse(decryptedResult);
+    } catch {
+      this.sendResponse({
+        messageId: message.messageId,
+        version: NativeMessagingVersion.Latest,
+        payload: {
+          error: "cannot-decrypt",
+        },
+      });
+      return;
+    }
   }
 
   private async sendEncryptedResponse(
@@ -218,4 +231,19 @@ export class NativeMessageHandlerService {
   private sendResponse(response: EncryptedMessageResponse | UnencryptedMessageResponse) {
     ipcRenderer.send("nativeMessagingReply", response);
   }
+
+  private trimNullCharsFromMessage(message: string): string {
+    // Trim all null bytes padded at the end of messages. This happens with C encryption libraries.
+    for (let i = message.length - 1; i >= 0; i--) {
+      // char code 0 is null
+      if (message.charCodeAt(i) === 0) {
+        message = message.substring(0, message.length - 1);
+      }
+      // char code 125 is } and 93 is ] which are valid json ending characters, stop checking
+      else if (message.charCodeAt(i) === 125 || message.charCodeAt(i) === 93) {
+        break;
+      }
+    }
+    return message;
+  }
 }